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

Unified Diff: Source/modules/geolocation/testing/GeolocationClientMock.cpp

Issue 490143002: Move GeolocationClient to the Oilpan heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Provide default empty impl of WebGeolocationClient::geolocationDestroyed() virtual Created 6 years, 4 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/testing/GeolocationClientMock.cpp
diff --git a/Source/modules/geolocation/testing/GeolocationClientMock.cpp b/Source/modules/geolocation/testing/GeolocationClientMock.cpp
index 7c281b6ba601a6b00bda02ddc0120afdbff96cbe..d613d973a8fd63d64150e1bd9a432875541c7bbf 100644
--- a/Source/modules/geolocation/testing/GeolocationClientMock.cpp
+++ b/Source/modules/geolocation/testing/GeolocationClientMock.cpp
@@ -126,11 +126,6 @@ void GeolocationClientMock::permissionTimerFired(Timer<GeolocationClientMock>* t
m_pendingPermissions.clear();
}
-void GeolocationClientMock::geolocationDestroyed()
-{
- ASSERT(!m_isActive);
-}
-
void GeolocationClientMock::startUpdating()
{
ASSERT(!m_isActive);
@@ -184,4 +179,14 @@ void GeolocationClientMock::clearError()
m_errorMessage = String();
}
+void GeolocationClientMock::trace(Visitor* visitor)
+{
+#if ENABLE(OILPAN)
+ visitor->trace(m_controllers);
+#endif
+ visitor->trace(m_lastPosition);
+ visitor->trace(m_pendingPermissions);
+ GeolocationClient::trace(visitor);
+}
+
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698