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

Issue 149506: GTK: For the render view, count keyboard grab shadows the same as focus event... (Closed)

Created:
11 years, 5 months ago by Evan Stade
Modified:
9 years, 6 months ago
Reviewers:
tony
CC:
chromium-reviews_googlegroups.com, darin (slow to review), brettw, jam, Ben Goodger (Google)
Visibility:
Public.

Description

GTK: For the render view, count keyboard grab shadows the same as focus events. This fixes an age old bug that dropdown menus still looked depressed after you select something from them and they collapse. It also fixes the more recent bug that you could have focus in the render view without any indication drawn by the renderer. I suspect there was a change to webkit that exacerbated this bug, because it seems like something we would have noticed earlier. Verified in a debugger that showing/hiding a dropdown menu doesn't emit focus signals, but does emit grab notify signals. Also verified that the bug is no longer reproducible in a simple page with <select> and <input> elements. BUG=16313 TEST=see desc. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=20574

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -0 lines) Patch
M chrome/browser/renderer_host/render_widget_host_view_gtk.cc View 4 chunks +18 lines, -0 lines 1 comment Download

Messages

Total messages: 2 (0 generated)
Evan Stade
http://codereview.chromium.org/149506/diff/1/2 File chrome/browser/renderer_host/render_widget_host_view_gtk.cc (right): http://codereview.chromium.org/149506/diff/1/2#newcode223 Line 223: if (was_grabbed) this check seems backward, but is ...
11 years, 5 months ago (2009-07-11 01:14:06 UTC) #1
tony
11 years, 5 months ago (2009-07-13 17:34:55 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698