Audit the last usage of Geolocation and Notification permissions.

chrome/browser/notifications/desktop_notification_service.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/notifications/desktop_notification_service.h"
 
 #include "base/bind.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/scoped_user_pref_update.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 494 matching lines @@
     const GURL& origin,
     content::RenderFrameHost* render_frame_host,
     const base::Closure& callback) {
   // If |origin| hasn't been seen before and the default content setting for
   // notifications is "ask", show an infobar.
   // The cache can only answer queries on the IO thread once it's initialized,
   // so don't ask the cache.
   WebContents* web_contents = WebContents::FromRenderFrameHost(
       render_frame_host);
   ContentSetting setting = GetContentSetting(origin);
+
+  if (setting == CONTENT_SETTING_ALLOW) {

[dewittj, 2014/06/26 23:17:47]

I wonder if this would be better done in ChromeContentBrowserClient (if you are
only interested in web notifications). 
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/chr...

[dewittj, 2014/06/27 02:52:03]

In other words, I'm wary of a special exception right here instead of an API for
content settings that's going to guarantee that the right metrics are recorded.

[Daniel Nishi, 2014/06/27 17:25:19]

I've moved this into the HostContentSettingsMap to avoid having special
exceptions.

+    profile_->GetHostContentSettingsMap()->UpdateLastUsage(
+        origin, origin, CONTENT_SETTINGS_TYPE_NOTIFICATIONS);
+  }
+
   if (setting == CONTENT_SETTING_ASK) {
     if (PermissionBubbleManager::Enabled()) {
       PermissionBubbleManager* bubble_manager =
           PermissionBubbleManager::FromWebContents(web_contents);
       if (bubble_manager) {
         bubble_manager->AddRequest(new NotificationPermissionRequest(
             this,
             origin,
             DisplayNameForOriginInProcessId(
                 origin, render_frame_host->GetProcess()->GetID()),
@@ skipping 227 matching lines @@
   // Tell the IO thread that this extension's permission for notifications
   // has changed.
   extensions::InfoMap* extension_info_map =
       extensions::ExtensionSystem::Get(profile_)->info_map();
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&extensions::InfoMap::SetNotificationsDisabled,
                  extension_info_map, notifier_id.id, !enabled));
 #endif
 }