Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2075)

Unified Diff: Source/modules/geolocation/GeolocationController.cpp

Issue 255983003: Oilpan: Move all supplements of Page, Document, and WorkerClients to the managed heap. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: using namespace WebCore Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/modules/geolocation/GeolocationController.cpp
diff --git a/Source/modules/geolocation/GeolocationController.cpp b/Source/modules/geolocation/GeolocationController.cpp
index ae5bc6089d5663066242592240d88d707d91bc49..e941fe4356cbb4418aaa7d9188dd0ab34cbc9880 100644
--- a/Source/modules/geolocation/GeolocationController.cpp
+++ b/Source/modules/geolocation/GeolocationController.cpp
@@ -74,9 +74,9 @@ void GeolocationController::willBeDestroyed()
m_client->geolocationDestroyed();
}
-PassOwnPtr<GeolocationController> GeolocationController::create(Page& page, GeolocationClient* client)
+PassOwnPtrWillBeRawPtr<GeolocationController> GeolocationController::create(Page& page, GeolocationClient* client)
{
- return adoptPtr(new GeolocationController(page, client));
+ return adoptPtrWillBeNoop(new GeolocationController(page, client));
}
void GeolocationController::addObserver(Geolocation* observer, bool enableHighAccuracy)
@@ -179,9 +179,16 @@ const char* GeolocationController::supplementName()
return "GeolocationController";
}
+void GeolocationController::trace(Visitor* visitor)
+{
+ visitor->trace(m_lastPosition);
+ visitor->trace(m_observers);
+ visitor->trace(m_highAccuracyObservers);
+}
+
void provideGeolocationTo(Page& page, GeolocationClient* client)
{
- Supplement<Page>::provideTo(page, GeolocationController::supplementName(), GeolocationController::create(page, client));
+ WillBeHeapSupplement<Page>::provideTo(page, GeolocationController::supplementName(), GeolocationController::create(page, client));
}
} // namespace WebCore
« no previous file with comments | « Source/modules/geolocation/GeolocationController.h ('k') | Source/modules/navigatorcontentutils/NavigatorContentUtils.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698