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

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

Issue 6693054: Get rid of extensions dependency from content\browser. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 9 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_page_actions_module.cc
===================================================================
--- chrome/browser/extensions/extension_page_actions_module.cc (revision 80243)
+++ chrome/browser/extensions/extension_page_actions_module.cc (working copy)
@@ -9,6 +9,7 @@
#include "base/string_number_conversions.h"
#include "chrome/browser/browser_list.h"
#include "chrome/browser/extensions/extension_page_actions_module_constants.h"
+#include "chrome/browser/extensions/extension_tab_helper.h"
#include "chrome/browser/extensions/extension_tabs_module.h"
#include "chrome/browser/extensions/extension_service.h"
#include "chrome/browser/profiles/profile.h"
@@ -89,7 +90,7 @@
page_action->SetIsVisible(tab_id, enable);
page_action->SetTitle(tab_id, title);
page_action->SetIconIndex(tab_id, icon_id);
- contents->tab_contents()->PageActionStateChanged();
+ contents->extension_tab_helper()->PageActionStateChanged();
return true;
}
@@ -111,7 +112,7 @@
kNoTabError, base::IntToString(tab_id));
return false;
}
- contents_ = wrapper->tab_contents();
+ contents_ = wrapper;
return true;
}
@@ -123,7 +124,7 @@
return false;
page_action_->SetIsVisible(tab_id, visible);
- contents_->PageActionStateChanged();
+ contents_->extension_tab_helper()->PageActionStateChanged();
return true;
}
@@ -175,7 +176,7 @@
EXTENSION_FUNCTION_VALIDATE(false);
}
- contents_->PageActionStateChanged();
+ contents_->extension_tab_helper()->PageActionStateChanged();
return true;
}
@@ -192,7 +193,7 @@
EXTENSION_FUNCTION_VALIDATE(args->GetString("title", &title));
page_action_->SetTitle(tab_id, title);
- contents_->PageActionStateChanged();
+ contents_->extension_tab_helper()->PageActionStateChanged();
return true;
}
@@ -215,7 +216,7 @@
popup_url = GetExtension()->GetResourceURL(popup_string);
page_action_->SetPopupUrl(tab_id, popup_url);
- contents_->PageActionStateChanged();
+ contents_->extension_tab_helper()->PageActionStateChanged();
return true;
}
@@ -241,7 +242,7 @@
SkColor color = SkColorSetARGB(color_array[3], color_array[0], color_array[1],
color_array[2]);
page_action_->SetBadgeBackgroundColor(tab_id, color);
- contents_->PageActionStateChanged();
+ contents_->extension_tab_helper()->PageActionStateChanged();
return true;
}
@@ -267,7 +268,7 @@
SkColor color = SkColorSetARGB(color_array[3], color_array[0], color_array[1],
color_array[2]);
page_action_->SetBadgeTextColor(tab_id, color);
- contents_->PageActionStateChanged();
+ contents_->extension_tab_helper()->PageActionStateChanged();
return true;
}
@@ -286,6 +287,6 @@
EXTENSION_FUNCTION_VALIDATE(args->GetString("text", &text));
page_action_->SetBadgeText(tab_id, text);
- contents_->PageActionStateChanged();
+ contents_->extension_tab_helper()->PageActionStateChanged();
return true;
}
« no previous file with comments | « chrome/browser/extensions/extension_page_actions_module.h ('k') | chrome/browser/extensions/extension_startup_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698