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

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: adjust a comment 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 8e6dff68f781c09c66bcc9e570dea23f4e085ae7..f92e3a1f37d5d8bc6579496090d33117ee0970a8 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