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

Unified Diff: chrome/renderer/render_view.cc

Issue 208020: Change the view mode when switching between moles and toolstrips, and (Closed)
Patch Set: build system workarounds Created 11 years, 3 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
« no previous file with comments | « chrome/renderer/render_view.h ('k') | chrome/renderer/renderer_resources.grd » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/render_view.cc
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc
index 2e68db7f54d9d77c8d0245a2fe61d0276be2c3bf..a1aea97cf6003d7e93748fe1638a23c4f90d906f 100644
--- a/chrome/renderer/render_view.cc
+++ b/chrome/renderer/render_view.cc
@@ -2080,6 +2080,11 @@ void RenderView::didCreateDocumentElement(WebFrame* frame) {
RenderThread::current()->user_script_slave()->InjectScripts(
frame, UserScript::DOCUMENT_START);
}
+ if (view_type_ == ViewType::EXTENSION_TOOLSTRIP ||
+ view_type_ == ViewType::EXTENSION_MOLE) {
+ InjectToolstripCSS();
+ ExtensionProcessBindings::SetViewType(webview(), view_type_);
+ }
while (!pending_code_execution_queue_.empty()) {
scoped_refptr<CodeExecutionInfo> info =
@@ -2974,6 +2979,15 @@ void RenderView::OnMediaPlayerActionAt(int x,
}
void RenderView::OnNotifyRendererViewType(ViewType::Type type) {
+ // When this is first set, the bindings aren't fully loaded. We only need
+ // to call through this API after the page has already been loaded. It's
+ // also called in didCreateDocumentElement to bootstrap.
+ if (view_type_ != ViewType::INVALID) {
+ if (type == ViewType::EXTENSION_MOLE ||
+ type == ViewType::EXTENSION_TOOLSTRIP) {
+ ExtensionProcessBindings::SetViewType(webview(), type);
+ }
+ }
view_type_ = type;
}
@@ -3249,6 +3263,17 @@ void RenderView::OnExtensionResponse(int request_id,
request_id, success, response, error);
}
+void RenderView::InjectToolstripCSS() {
+ if (view_type_ != ViewType::EXTENSION_TOOLSTRIP)
+ return;
+
+ static const base::StringPiece toolstrip_css(
+ ResourceBundle::GetSharedInstance().GetRawDataResource(
+ IDR_EXTENSION_TOOLSTRIP_CSS));
+ std::string css = toolstrip_css.as_string();
+ InsertCSS(L"", css, "ToolstripDefaultCSS");
+}
+
void RenderView::OnExtensionMessageInvoke(const std::string& function_name,
const ListValue& args) {
RendererExtensionBindings::Invoke(function_name, args, this);
« no previous file with comments | « chrome/renderer/render_view.h ('k') | chrome/renderer/renderer_resources.grd » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698