PS - Filtering activeTab URL

chrome/browser/extensions/extension_tab_util.cc

Index: chrome/browser/extensions/extension_tab_util.cc
diff --git a/chrome/browser/extensions/extension_tab_util.cc b/chrome/browser/extensions/extension_tab_util.cc
index ffaad663823c3a21a005a7035048a64def64531b..3c51503a42529d026967ca9cb7970422487a7ed4 100644
--- a/chrome/browser/extensions/extension_tab_util.cc
+++ b/chrome/browser/extensions/extension_tab_util.cc
@@ -126,6 +126,13 @@ ExtensionTabUtil::OpenTabParams::OpenTabParams()
 ExtensionTabUtil::OpenTabParams::~OpenTabParams() {
 }
 
+// static
+void ExtensionTabUtil::PlatformSpecificFiltering(
+    const std::string& extension_id, api::tabs::Tab* tab) {
+  if (g_delegate)
+    g_delegate->ScrubTabForExtension(extension_id, tab);

[Ivan Šandrk, 2017/05/03 13:00:51]

PlatformSpecificFiltering vs. ScrubTabForExtension

Seems as now I have a naming inconsistency maybe? Anyway I think I prefer the
name PlatformSpecificFiltering. WDYT?

[Devlin, 2017/05/03 15:14:15]

I don't feel strongly on the naming, but see also comment in
extension_action_api.cc that would make this method unnecessary.

+}
+
 // Opens a new tab for a given extension. Returns NULL and sets |error| if an
 // error occurs.
 base::DictionaryValue* ExtensionTabUtil::OpenTab(
@@ -488,8 +495,7 @@ void ExtensionTabUtil::ScrubTabForExtension(const Extension* extension,
     tab->title.reset();
     tab->fav_icon_url.reset();
   }
-  if (g_delegate)
-    g_delegate->ScrubTabForExtension(extension, contents, tab);
+  PlatformSpecificFiltering(extension ? extension->id() : "", tab);
 }
 
 bool ExtensionTabUtil::GetTabStripModel(const WebContents* web_contents,