Tested your macro, it does loop correctly. Try to debug it using key F8 to step through the code to see what happens.
Alt+F8 to open "Run Macro" then use "Step Into" and use F8 to advance line by line
or Alt+F11 to goto VBE then use F8.