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

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: fix up unit test 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
« no previous file with comments | « content/browser/frame_host/render_frame_host_impl.h ('k') | content/public/browser/permission_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 e6eb050896fbef0b0b43002ca613443e912bf69d..6ddcabd7149de7f32f0ba92fd7f78b25e39b38d8 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -2319,9 +2319,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 =
@@ -3171,19 +3169,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());
« no previous file with comments | « content/browser/frame_host/render_frame_host_impl.h ('k') | content/public/browser/permission_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698