I also don't see anything obviously wrong, but sometimes timing can be a problem. So, you might start by adding a Pause after each of the key actions:
This is probably excessive, but the first step is to get the macro working correctly. Then you can try reducing/eliminating some of the Pauses.
If your intent is to open a specified file, then this macro will get the job done, and is simpler and more robust:
<img src="/uploads/default/original/2X/a/ac9b96ac02fabbfeae5e11b9a477bf7021755dea.png" width="444" height="129">
If the file extension is not the default for your app, then you will need to change "Default Application" to your app.