Oilpan: have GeolocationInspectorAgent persistently track its controllers.

Source/modules/geolocation/testing/GeolocationClientMock.cpp

Index: Source/modules/geolocation/testing/GeolocationClientMock.cpp
diff --git a/Source/modules/geolocation/testing/GeolocationClientMock.cpp b/Source/modules/geolocation/testing/GeolocationClientMock.cpp
index a3067e31d923e465891dfea4af4048dcc68b9b79..b82d2a94e820a524152862dfe3b348de996cf83a 100644
--- a/Source/modules/geolocation/testing/GeolocationClientMock.cpp
+++ b/Source/modules/geolocation/testing/GeolocationClientMock.cpp
@@ -167,13 +167,13 @@ void GeolocationClientMock::controllerTimerFired(Timer<GeolocationClientMock>* t
     ASSERT_UNUSED(timer, timer == &m_controllerTimer);
 
     // Make a copy of the set of controllers since it might be modified while iterating.
-    HashSet<GeolocationController*> controllers = m_controllers;
+    GeolocationControllers controllers = m_controllers;
     if (m_lastPosition.get()) {
         ASSERT(!m_hasError);
-        for (HashSet<GeolocationController*>::iterator it = controllers.begin(); it != controllers.end(); ++it)
+        for (GeolocationControllers::iterator it = controllers.begin(); it != controllers.end(); ++it)
             (*it)->positionChanged(m_lastPosition.get());
     } else if (m_hasError) {
-        for (HashSet<GeolocationController*>::iterator it = controllers.begin(); it != controllers.end(); ++it)
+        for (GeolocationControllers::iterator it = controllers.begin(); it != controllers.end(); ++it)
             (*it)->errorOccurred(GeolocationError::create(GeolocationError::PositionUnavailable, m_errorMessage));
     }
 }