Add hooks to permission layer for permission action reporting

chrome/browser/safe_browsing/ui_manager.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/safe_browsing/ui_manager.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/debug/leak_tracker.h"
@@ skipping 263 matching lines @@
     const base::Closure& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   BrowserThread::PostTaskAndReply(
       BrowserThread::IO, FROM_HERE,
       base::Bind(
           &SafeBrowsingUIManager::ReportInvalidCertificateChainOnIOThread, this,
           serialized_report),
       callback);
 }
 
+void SafeBrowsingUIManager::ReportPermissionAction(
+    const GURL& origin,
+    content::PermissionType permission,
+    PermissionAction action) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+
+  // Do not report permission action when the permission action reporting
+  // flag is not enabled.
+  if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnablePermissionActionReporting)) {
+    return;
+  }
+
+  Profile* profile = ProfileManager::GetLastUsedProfile();

[raymes, 2016/06/16 01:10:23]

We should use the actual profile (as discussed).

[stefanocs, 2016/06/16 01:52:20]

Done.

+
+  // Do not report when SafeBrowsing is not enabled.
+  if (!profile->GetPrefs()->GetBoolean(prefs::kSafeBrowsingEnabled)) {

[raymes, 2016/06/16 01:10:22]

Let's check with nparker for this check.

[raymes, 2016/06/16 01:10:23]

nit: no {} for single-line if statements (and below)

[stefanocs, 2016/06/16 01:52:20]

Acknowledged.

[stefanocs, 2016/06/16 01:52:20]

Done.

+    return;
+  }
+  // Do not report if profile doesn't have profile sync service.
+  if (!ProfileSyncServiceFactory::HasProfileSyncService(profile)) {
+    return;
+  }
+
+  ProfileSyncService* profile_sync_service =
+      ProfileSyncServiceFactory::GetForProfile(profile);
+  syncer::ModelTypeSet preferred_data_types =
+      profile_sync_service->GetPreferredDataTypes();
+
+  // Do not report if profile can't start sync immediately.
+  if (!profile_sync_service->CanSyncStart()) {

[raymes, 2016/06/16 01:10:23]

Let's check with pavely about how to find out:
-Whether sync is enabled
-Whether tab sync is enabled
-Whether pref sync is enabled

[stefanocs, 2016/06/16 01:52:20]

Acknowledged.

+    return;
+  }
+  // Do not report if Chrome Tab Sync is not enabled.
+  if (!preferred_data_types.Has(syncer::PROXY_TABS)) {
+    return;
+  }
+  // Do not report if Chrome Pref Sync is not enabled.
+  if (!preferred_data_types.Has(syncer::PREFERENCES)) {
+    return;
+  }
+
+  BrowserThread::PostTask(
+      BrowserThread::IO, FROM_HERE,
+      base::Bind(&SafeBrowsingUIManager::ReportPermissionActionOnIOThread, this,
+                 origin, permission, action));
+}
+
 void SafeBrowsingUIManager::AddObserver(Observer* observer) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   observer_list_.AddObserver(observer);
 }
 
 void SafeBrowsingUIManager::RemoveObserver(Observer* observer) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   observer_list_.RemoveObserver(observer);
 }
 
 void SafeBrowsingUIManager::ReportInvalidCertificateChainOnIOThread(
     const std::string& serialized_report) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   // The service may delete the ping manager (i.e. when user disabling service,
   // etc). This happens on the IO thread.
   if (!sb_service_ || !sb_service_->ping_manager())
     return;
 
   sb_service_->ping_manager()->ReportInvalidCertificateChain(serialized_report);
 }
 
+void SafeBrowsingUIManager::ReportPermissionActionOnIOThread(
+    const GURL& origin,
+    content::PermissionType permission,
+    PermissionAction action) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+
+  // The service may delete the ping manager (i.e. when user disabling service,
+  // etc). This happens on the IO thread.
+  if (!sb_service_ || !sb_service_->ping_manager())
+    return;
+
+  sb_service_->ping_manager()->ReportPermissionAction(origin, permission,
+                                                      action);
+}
+
 // If the user had opted-in to send ThreatDetails, this gets called
 // when the report is ready.
 void SafeBrowsingUIManager::SendSerializedThreatDetails(
     const std::string& serialized) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   // The service may delete the ping manager (i.e. when user disabling service,
   // etc). This happens on the IO thread.
   if (sb_service_.get() == NULL || sb_service_->ping_manager() == NULL)
     return;
@@ skipping 49 matching lines @@
   }
 
   WhitelistUrlSet* site_list =
       static_cast<WhitelistUrlSet*>(web_contents->GetUserData(kWhitelistKey));
   if (!site_list)
     return false;
   return site_list->Contains(maybe_whitelisted_url);
 }
 
 }  // namespace safe_browsing