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

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

Issue 923463003: [Extensions] Remove the Infobar API (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments Created 5 years, 10 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_view_host.cc
diff --git a/chrome/browser/extensions/extension_view_host.cc b/chrome/browser/extensions/extension_view_host.cc
index 4650159daafdd8731a01d19e607950481e75d9fe..40e88d6c4e45242f0ed00ee3d94659205e0870e1 100644
--- a/chrome/browser/extensions/extension_view_host.cc
+++ b/chrome/browser/extensions/extension_view_host.cc
@@ -63,7 +63,6 @@ ExtensionViewHost::ExtensionViewHost(
associated_web_contents_(NULL) {
// Not used for panels, see PanelHost.
DCHECK(host_type == VIEW_TYPE_EXTENSION_DIALOG ||
- host_type == VIEW_TYPE_EXTENSION_INFOBAR ||
host_type == VIEW_TYPE_EXTENSION_POPUP);
}
@@ -105,13 +104,6 @@ void ExtensionViewHost::OnDidStopLoading() {
view_->DidStopLoading();
}
-void ExtensionViewHost::OnDocumentAvailable() {
- if (extension_host_type() == VIEW_TYPE_EXTENSION_INFOBAR) {
- // No style sheet for other types, at the moment.
- InsertInfobarCSS();
- }
-}
-
void ExtensionViewHost::LoadInitialURL() {
if (!ExtensionSystem::Get(browser_context())->
runtime_data()->IsBackgroundPageReady(extension())) {
@@ -314,12 +306,4 @@ void ExtensionViewHost::Observe(int type,
ExtensionHost::Observe(type, source, details);
}
-void ExtensionViewHost::InsertInfobarCSS() {
- static const base::StringPiece css(
- ResourceBundle::GetSharedInstance().GetRawDataResource(
- IDR_EXTENSIONS_INFOBAR_CSS));
-
- host_contents()->InsertCSS(css.as_string());
-}
-
} // namespace extensions
« no previous file with comments | « chrome/browser/extensions/extension_view_host.h ('k') | chrome/browser/extensions/extension_view_host_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698