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

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

Issue 171333003: Pass implementation object to supplemental classes by reference (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase Created 6 years, 10 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/InternalsGeolocation.cpp
diff --git a/Source/modules/geolocation/testing/InternalsGeolocation.cpp b/Source/modules/geolocation/testing/InternalsGeolocation.cpp
index 6174a849c0ceb9f6375e29d033b4c3b1ba6aec6f..ff1f1b099563782fb9fe48432e63b86895229851 100644
--- a/Source/modules/geolocation/testing/InternalsGeolocation.cpp
+++ b/Source/modules/geolocation/testing/InternalsGeolocation.cpp
@@ -41,45 +41,45 @@
namespace WebCore {
-void InternalsGeolocation::setGeolocationClientMock(Internals* internals, Document* document)
+void InternalsGeolocation::setGeolocationClientMock(Internals&, Document* document)
{
- ASSERT(internals && document && document->page());
+ ASSERT(document && document->page());
GeolocationController* controller = GeolocationController::from(document->page());
GeolocationClientMock* client = new GeolocationClientMock();
controller->setClientForTest(client);
client->setController(controller);
}
-void InternalsGeolocation::setGeolocationPosition(Internals* internals, Document* document, double latitude, double longitude, double accuracy)
+void InternalsGeolocation::setGeolocationPosition(Internals&, Document* document, double latitude, double longitude, double accuracy)
{
- ASSERT(internals && document && document->page());
+ ASSERT(document && document->page());
GeolocationClientMock* client = geolocationClient(document);
if (!client)
return;
client->setPosition(GeolocationPosition::create(currentTime(), latitude, longitude, accuracy));
}
-void InternalsGeolocation::setGeolocationPositionUnavailableError(Internals* internals, Document* document, const String& message)
+void InternalsGeolocation::setGeolocationPositionUnavailableError(Internals&, Document* document, const String& message)
{
- ASSERT(internals && document && document->page());
+ ASSERT(document && document->page());
GeolocationClientMock* client = geolocationClient(document);
if (!client)
return;
client->setPositionUnavailableError(message);
}
-void InternalsGeolocation::setGeolocationPermission(Internals* internals, Document* document, bool allowed)
+void InternalsGeolocation::setGeolocationPermission(Internals&, Document* document, bool allowed)
{
- ASSERT(internals && document && document->page());
+ ASSERT(document && document->page());
GeolocationClientMock* client = geolocationClient(document);
if (!client)
return;
client->setPermission(allowed);
}
-int InternalsGeolocation::numberOfPendingGeolocationPermissionRequests(Internals* internals, Document* document)
+int InternalsGeolocation::numberOfPendingGeolocationPermissionRequests(Internals&, Document* document)
{
- ASSERT(internals && document && document->page());
+ ASSERT(document && document->page());
GeolocationClientMock* client = geolocationClient(document);
if (!client)
return -1;

Powered by Google App Engine
This is Rietveld 408576698