diff options
| author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-14 16:45:05 +0000 |
|---|---|---|
| committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-14 16:45:05 +0000 |
| commit | 033970a1a2aabb6853057cd16cd90aac1d9e4a9c (patch) | |
| tree | 27cfe8126590632f7d99182c094ca2cdba32aa88 /chrome/renderer | |
| parent | ac0efda36e8f20bc74364e6e74e39f451a482537 (diff) | |
| download | chromium_src-033970a1a2aabb6853057cd16cd90aac1d9e4a9c.zip chromium_src-033970a1a2aabb6853057cd16cd90aac1d9e4a9c.tar.gz chromium_src-033970a1a2aabb6853057cd16cd90aac1d9e4a9c.tar.bz2 | |
The focus ring was not shown on Linux toolkit views.
This is because on the renderer side the ring color was set to a value received
from the browser, but on Linux toolkit view we are not initializing that value in
the browser (just like we do on Windows, we keep the default WebKit value).
So it would be 0, and the rings would not be shown.
BUG=None
TEST=Open www.google.com in Chrome toolkit views. Move the focus around with
the tab key. A yellowish focus ring should show which node has focus.
Review URL: http://codereview.chromium.org/270099
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28968 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer')
| -rw-r--r-- | chrome/renderer/render_view.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc index 83e98a0..f4618da 100644 --- a/chrome/renderer/render_view.cc +++ b/chrome/renderer/render_view.cc @@ -3030,7 +3030,7 @@ void RenderView::OnEnableIntrinsicWidthChangedMode() { void RenderView::OnSetRendererPrefs(const RendererPreferences& renderer_prefs) { renderer_preferences_ = renderer_prefs; UpdateFontRenderingFromRendererPrefs(); -#if defined(OS_LINUX) +#if defined(OS_LINUX) && !defined(TOOLKIT_VIEWS) WebColorName name = WebKit::WebColorWebkitFocusRingColor; WebKit::setNamedColors(&name, &renderer_prefs.focus_ring_color, 1); #endif |