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

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

Issue 10392142: Revert 137630 - Broke ASAN (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 7 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.cc
===================================================================
--- chrome/browser/extensions/extension_tab_helper.cc (revision 137631)
+++ chrome/browser/extensions/extension_tab_helper.cc (working copy)
@@ -5,8 +5,6 @@
#include "chrome/browser/extensions/extension_tab_helper.h"
#include "chrome/browser/extensions/extension_service.h"
-#include "chrome/browser/extensions/page_action_controller.h"
-#include "chrome/browser/extensions/script_executor_impl.h"
#include "chrome/browser/extensions/webstore_inline_installer.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/sessions/restore_tab_helper.h"
@@ -29,8 +27,6 @@
#include "ui/gfx/image/image.h"
using content::WebContents;
-using extensions::ScriptExecutorImpl;
-using extensions::PageActionController;
namespace {
@@ -45,8 +41,6 @@
ALLOW_THIS_IN_INITIALIZER_LIST(
extension_function_dispatcher_(wrapper->profile(), this)),
wrapper_(wrapper) {
- script_executor_.reset(new ScriptExecutorImpl(wrapper->web_contents()));
- action_box_controller_.reset(new PageActionController(wrapper, this));
}
ExtensionTabHelper::~ExtensionTabHelper() {
@@ -58,26 +52,14 @@
}
void ExtensionTabHelper::PageActionStateChanged() {
- // TODO(kalman): replace this with just the Observer interface.
web_contents()->NotifyNavigationStateChanged(
content::INVALIDATE_TYPE_PAGE_ACTIONS);
-
- FOR_EACH_OBSERVER(Observer, observers_, OnPageActionStateChanged());
}
void ExtensionTabHelper::GetApplicationInfo(int32 page_id) {
Send(new ExtensionMsg_GetApplicationInfo(routing_id(), page_id));
}
-void ExtensionTabHelper::AddObserver(ExtensionTabHelper::Observer* observer) {
- observers_.AddObserver(observer);
-}
-
-void ExtensionTabHelper::RemoveObserver(
- ExtensionTabHelper::Observer* observer) {
- observers_.RemoveObserver(observer);
-}
-
void ExtensionTabHelper::SetExtensionApp(const Extension* extension) {
DCHECK(!extension || extension->GetFullLaunchURL().is_valid());
extension_app_ = extension;
« no previous file with comments | « chrome/browser/extensions/extension_tab_helper.h ('k') | chrome/browser/extensions/extension_tabs_module.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698