Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(554)

Unified Diff: chrome/browser/extensions/extension_tab_helper.h

Issue 10443105: Take 2 at implementing activeTab. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: many more tests Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_tab_helper.h
diff --git a/chrome/browser/extensions/extension_tab_helper.h b/chrome/browser/extensions/extension_tab_helper.h
index 66ed9f1bda94399316223517bb0fe9741d088cb7..166491ee459869a7af1520d1d0cf9193d90a6f78 100644
--- a/chrome/browser/extensions/extension_tab_helper.h
+++ b/chrome/browser/extensions/extension_tab_helper.h
@@ -9,6 +9,7 @@
#include "base/memory/ref_counted.h"
#include "base/memory/weak_ptr.h"
#include "base/observer_list.h"
+#include "chrome/browser/extensions/active_tab_permission_manager.h"
#include "chrome/browser/extensions/app_notify_channel_setup.h"
#include "chrome/browser/extensions/extension_function_dispatcher.h"
#include "chrome/browser/extensions/image_loading_tracker.h"
@@ -59,6 +60,12 @@ class ExtensionTabHelper
// the data is available.
void GetApplicationInfo(int32 page_id);
+ // Gets the ID of the tab.
+ int GetTabId() const;
+
+ // Gets the window ID of the tab.
+ int GetWindowId() const;
+
// App extensions ------------------------------------------------------------
// Sets the extension denoting this as an app. If |extension| is non-null this
@@ -103,12 +110,18 @@ class ExtensionTabHelper
extensions::LocationBarController* location_bar_controller();
+ extensions::ActiveTabPermissionManager* active_tab_permission_manager() {
+ return &active_tab_permission_manager_;
+ }
+
// Sets a non-extension app icon associated with WebContents and fires an
// INVALIDATE_TYPE_TITLE navigation state change to trigger repaint of title.
void SetAppIcon(const SkBitmap& app_icon);
private:
// content::WebContentsObserver overrides.
+ virtual void RenderViewCreated(content::RenderViewHost* render_view_host)
Aaron Boodman 2012/06/08 05:31:30 Weird to have OVERRIDE on line by itself. Consider
not at google - send to devlin 2012/06/12 20:40:51 ok.
+ OVERRIDE;
virtual void DidNavigateMainFrame(
const content::LoadCommittedDetails& details,
const content::FrameNavigateParams& params) OVERRIDE;
@@ -197,6 +210,8 @@ class ExtensionTabHelper
scoped_ptr<extensions::LocationBarController> location_bar_controller_;
scoped_refptr<extensions::ScriptBadgeController> script_badge_controller_;
+ extensions::ActiveTabPermissionManager active_tab_permission_manager_;
+
DISALLOW_COPY_AND_ASSIGN(ExtensionTabHelper);
};

Powered by Google App Engine
This is Rietveld 408576698