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

Unified Diff: chrome/renderer/render_view.cc

Issue 173642: Set the focus ring color to match the Gtk theme focus color. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: connect to notification Created 11 years, 4 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/renderer/render_view.cc
===================================================================
--- chrome/renderer/render_view.cc (revision 25076)
+++ chrome/renderer/render_view.cc (working copy)
@@ -299,14 +299,14 @@
decrement_shared_popup_at_destruction_ = false;
}
- OnSetRendererPrefs(renderer_prefs);
-
devtools_agent_.reset(new DevToolsAgent(routing_id, this));
webwidget_ = WebView::Create();
webkit_preferences_.Apply(webview());
webview()->InitializeMainFrame(this);
+ OnSetRendererPrefs(renderer_prefs);
+
#if defined(OS_LINUX)
// We have to enable ourselves as the editor delegate on linux so we can copy
// text selections to the X clipboard.
@@ -2775,6 +2775,10 @@
void RenderView::OnSetRendererPrefs(const RendererPreferences& renderer_prefs) {
renderer_preferences_ = renderer_prefs;
UpdateFontRenderingFromRendererPrefs();
+ webview()->SetThemeFocusRingColor(renderer_prefs.focus_ring_color_r,
+ renderer_prefs.focus_ring_color_g,
+ renderer_prefs.focus_ring_color_b);
+
}
void RenderView::OnMediaPlayerActionAt(int x,

Powered by Google App Engine
This is Rietveld 408576698