Partial revert 'for_each -> range-based'

There was a subtle change in semantics, the earlier loop was only
iterating until aEnd, covering only unique elements of the vector.

This partially reverts commit c04fd824

Change-Id: I75c5f1b343c3fa8855f6d7f8f706b9259bc9f6df

canvas/source/tools/spriteredrawmanager.cxx

@@ -265,9 +265,10 @@ namespace canvas
         // for each unique sprite, check the change event vector,
         // calculate the update operation from that, and add the
         // result to the aUpdateArea.
-        SpriteUpdater aSpriteUpdater( rUpdateAreas, maChangeRecords);
-        for (auto const& aUpdatableSprite : aUpdatableSprites)
-            aSpriteUpdater( aUpdatableSprite);
+        ::std::for_each( aUpdatableSprites.begin(),
+                         aEnd,
+                         SpriteUpdater( rUpdateAreas,
+                                        maChangeRecords) );
 
         // TODO(P2): Implement your own output iterator adapter, to
         // avoid that totally superfluous temp aUnchangedSprites