Hook the activeTab permission up to the omnibox, page action, and context menu APIs.

chrome/browser/extensions/active_tab_permission_manager.h

Index: chrome/browser/extensions/active_tab_permission_manager.h
diff --git a/chrome/browser/extensions/active_tab_permission_manager.h b/chrome/browser/extensions/active_tab_permission_manager.h
index 1dffe55139fa1a9c724c26d9caa0b587b86c891a..8f6d654c28d476593a3aefd4b41911da2c7d56ee 100644
--- a/chrome/browser/extensions/active_tab_permission_manager.h
+++ b/chrome/browser/extensions/active_tab_permission_manager.h
@@ -33,6 +33,9 @@ class ActiveTabPermissionManager : public content::WebContentsObserver,
   // permissions to it until the next page navigation or refresh.
   void GrantIfRequested(const Extension* extension);
 
+  // Like above, except make this find the Extension with |extension_id|.
+  void GrantIfRequested(const std::string& extension_id);

[Finnur, 2012/06/14 10:53:00]

I think the style guide doesn't like two functions using the same name when it
takes different params (exceptions made for simple cases like char* and
std::string). Maybe GrantByIdIfRequested?

[Aaron Boodman, 2012/06/14 20:50:45]

I think the styleguide allows for overloading in this way when the different
types are for convenience and the methods mean the same thing.

+
   // content::WebContentsObserver implementation.
   //
   // This is public to be called from ActiveTabTest as a way to fake creating