Fix InternalsGeolocation::setGeolocationClientMock to actually set the test...

Fix InternalsGeolocation::setGeolocationClientMock to actually set the test client for all GeolocationController.

This was caught by kenrb. It didn't manifest itself because it's a test-only method, and none of the geolocation tests which use this do so for an iframe.

Review URL: https://codereview.chromium.org/322583002

git-svn-id: svn://svn.chromium.org/blink/trunk@175907 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/modules/geolocation/testing/InternalsGeolocation.cpp

@@ -47,8 +47,8 @@ void InternalsGeolocation::setGeolocationClientMock(Internals&, Document* docume
     ASSERT(document && document->frame());
     GeolocationClientMock* client = new GeolocationClientMock();
 
-    for (LocalFrame* childFrame = document->page()->mainFrame(); childFrame; childFrame = childFrame->tree().nextSibling())
-        GeolocationController::from(document->frame())->setClientForTest(client);
+    for (LocalFrame* childFrame = document->page()->mainFrame(); childFrame; childFrame = childFrame->tree().traverseNext())
+        GeolocationController::from(childFrame)->setClientForTest(client);
 }
 
 void InternalsGeolocation::setGeolocationPosition(Internals&, Document* document, double latitude, double longitude, double accuracy)