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

Unified Diff: content/browser/frame_host/render_frame_host_impl.cc

Issue 2697473002: Remove last usage functions from HostContentSettingsMap and clean up prefs (Closed)
Patch Set: rebase address comments Created 3 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: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index 05bf13973d4ea29511853be2075fda1b88a4e29c..84afd7b005ac80b058a7d3298e6e3b617d69d1f6 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -2333,9 +2333,7 @@ void RenderFrameHostImpl::RegisterMojoInterfaces() {
// pointer.
GetInterfaceRegistry()->AddInterface(
base::Bind(&device::GeolocationServiceContext::CreateService,
- base::Unretained(geolocation_service_context),
- base::Bind(&RenderFrameHostImpl::DidUseGeolocationPermission,
- weak_ptr_factory_.GetWeakPtr())));
+ base::Unretained(geolocation_service_context)));
}
device::WakeLockServiceContext* wake_lock_service_context =
@@ -3185,19 +3183,6 @@ void RenderFrameHostImpl::SendNavigateMessage(
routing_id_, common_params, start_params, request_params));
}
-void RenderFrameHostImpl::DidUseGeolocationPermission() {
- PermissionManager* permission_manager =
- GetSiteInstance()->GetBrowserContext()->GetPermissionManager();
- if (!permission_manager)
- return;
-
- permission_manager->RegisterPermissionUsage(
- PermissionType::GEOLOCATION,
- last_committed_url().GetOrigin(),
- frame_tree_node()->frame_tree()->GetMainFrame()
- ->last_committed_url().GetOrigin());
-}
-
bool RenderFrameHostImpl::CanAccessFilesOfPageState(const PageState& state) {
return ChildProcessSecurityPolicyImpl::GetInstance()->CanReadAllFiles(
GetProcess()->GetID(), state.GetReferencedFiles());

Powered by Google App Engine
This is Rietveld 408576698