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

Unified Diff: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp

Issue 1367853002: Move GeolocationDispatcher into blink. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix layout tests Created 5 years, 3 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: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
index 086a79881696112c584d0d849db0724672006d89..e6746b6d1be97237ee60ceb10a67e2a09333dfc1 100644
--- a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
+++ b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
@@ -156,7 +156,6 @@
#include "core/timing/Performance.h"
#include "modules/app_banner/AppBannerController.h"
#include "modules/bluetooth/BluetoothSupplement.h"
-#include "modules/geolocation/GeolocationController.h"
#include "modules/notifications/NotificationPermissionClient.h"
#include "modules/permissions/PermissionController.h"
#include "modules/presentation/PresentationController.h"
@@ -219,7 +218,6 @@
#include "web/AssociatedURLLoader.h"
#include "web/CompositionUnderlineVectorBuilder.h"
#include "web/FindInPageCoordinates.h"
-#include "web/GeolocationClientProxy.h"
#include "web/LocalFileSystemClient.h"
#include "web/MIDIClientProxy.h"
#include "web/NavigatorContentUtilsClientImpl.h"
@@ -1659,7 +1657,6 @@ WebLocalFrameImpl::WebLocalFrameImpl(WebTreeScopeType scope, WebFrameClient* cli
, m_contentSettingsClient(0)
, m_inputEventsScaleFactorForEmulation(1)
, m_userMediaClientImpl(this)
- , m_geolocationClientProxy(GeolocationClientProxy::create(client ? client->geolocationClient() : 0))
, m_webDevToolsFrontend(0)
#if ENABLE(OILPAN)
, m_selfKeepAlive(this)
@@ -1689,7 +1686,6 @@ DEFINE_TRACE(WebLocalFrameImpl)
visitor->trace(m_devToolsAgent);
visitor->trace(m_textFinder);
visitor->trace(m_printContext);
- visitor->trace(m_geolocationClientProxy);
visitor->template registerWeakMembers<WebFrame, &WebFrame::clearWeakFrames>(this);
WebFrame::traceFrames(visitor, this);
}
@@ -1706,8 +1702,6 @@ void WebLocalFrameImpl::setCoreFrame(PassRefPtrWillBeRawPtr<LocalFrame> frame)
provideNotificationPermissionClientTo(*m_frame, NotificationPermissionClientImpl::create());
provideUserMediaTo(*m_frame, &m_userMediaClientImpl);
- provideGeolocationTo(*m_frame, m_geolocationClientProxy.get());
- m_geolocationClientProxy->setController(GeolocationController::from(m_frame.get()));
provideMIDITo(*m_frame, MIDIClientProxy::create(m_client ? m_client->webMIDIClient() : nullptr));
provideLocalFileSystemTo(*m_frame, LocalFileSystemClient::create());
provideNavigatorContentUtilsTo(*m_frame, NavigatorContentUtilsClientImpl::create(this));

Powered by Google App Engine
This is Rietveld 408576698