PS - Showing permission prompt for activeTab

chrome/browser/extensions/active_tab_permission_granter.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/extensions/active_tab_permission_granter.h"
 
 #include "chrome/browser/extensions/extension_action_runner.h"
 #include "chrome/browser/profiles/profile.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/navigation_handle.h"
@@ skipping 48 matching lines @@
       process_host->Send(create_message.Run(true));
       sent_to_hosts.insert(frame_host->GetProcess());
     }
   }
   // If the tab wasn't one of those processes already updated (it likely
   // wasn't), update it. Tabs don't need to update the origin whitelist.
   if (sent_to_hosts.count(tab_process) == 0)
     tab_process->Send(create_message.Run(false));
 }
 
+ActiveTabPermissionGranter::Delegate* g_delegate = nullptr;
+
 }  // namespace
 
 ActiveTabPermissionGranter::ActiveTabPermissionGranter(
     content::WebContents* web_contents,
     int tab_id,
     Profile* profile)
     : content::WebContentsObserver(web_contents),
       tab_id_(tab_id),
       extension_registry_observer_(this) {
   extension_registry_observer_.Add(ExtensionRegistry::Get(profile));
 }
 
 ActiveTabPermissionGranter::~ActiveTabPermissionGranter() {}
 
+// static
+void ActiveTabPermissionGranter::SetPlatformDelegate(Delegate* delegate) {
+  // Disallow setting it twice (but allow resetting - don't forget to free in
+  // that case).
+  CHECK(!g_delegate || !delegate);
+  g_delegate = delegate;
+}
+
+// static
+void ActiveTabPermissionGranter::FreePlatformDelegateForTesting() {
+  free(g_delegate);

[Devlin, 2017/05/23 19:28:03]

Why do we need this?  Just for ASAN?

[Ivan Šandrk, 2017/05/24 09:29:01]

Yes

+  g_delegate = nullptr;
+}
+
 void ActiveTabPermissionGranter::GrantIfRequested(const Extension* extension) {
   if (granted_extensions_.Contains(extension->id()))
     return;
 
   APIPermissionSet new_apis;
   URLPatternSet new_hosts;
 
   const PermissionsData* permissions_data = extension->permissions_data();
 
+  bool should_grant_active_tab = !g_delegate ||
+      g_delegate->ShouldGrantActiveTab(extension, web_contents());

[Devlin, 2017/05/23 19:28:04]

Is this git cl format'd?

[Ivan Šandrk, 2017/05/24 09:29:01]

Nope, now it is :)

   // If the extension requested all-hosts but has had it withheld, we grant it
   // active tab-style permissions, even if it doesn't have the activeTab
   // permission in the manifest.
-  if (permissions_data->HasAPIPermission(APIPermission::kActiveTab) ||
-      permissions_data->HasWithheldImpliedAllHosts()) {
+  if (should_grant_active_tab &&
+      (permissions_data->HasWithheldImpliedAllHosts() ||
+       permissions_data->HasAPIPermission(APIPermission::kActiveTab))) {
     new_hosts.AddOrigin(UserScript::ValidUserScriptSchemes(),
                         web_contents()->GetVisibleURL().GetOrigin());
     new_apis.insert(APIPermission::kTab);
   }
 
   if (permissions_data->HasAPIPermission(APIPermission::kTabCapture))
     new_apis.insert(APIPermission::kTabCaptureForTab);
 
   if (!new_apis.empty() || !new_hosts.is_empty()) {
     granted_extensions_.Insert(extension);
@@ skipping 95 matching lines @@
   CreateMessageFunction clear_message =
       base::Bind(&CreateClearMessage, extension_ids, tab_id_);
   SendMessageToProcesses(frame_hosts,
                          web_contents()->GetRenderProcessHost(),
                          clear_message);
 
   granted_extensions_.Clear();
 }
 
 }  // namespace extensions