Bug 23982

Summary: REGRESSION: Search field loses keyboard focus once a match is found
Product: WebKit Reporter: Geoffrey Garen <ggaren>
Component: Web Inspector (Deprecated)Assignee: Timothy Hatcher <timothy>
Status: RESOLVED FIXED    
Severity: Normal CC: aroben, kmccullough, timothy
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
Proposed Patch darin: review+

Geoffrey Garen
Reported 2009-02-16 16:12:33 PST
The Web Inspector's search field loses focus once a match is found. This makes it impossible to keep typing, so you often end up with something not very helpful selected. TO REPRODUCE: 1. Navigate to google.com 2. Start debugging. 3. In the Scripts pane's search field, type "ret" --> Notice that the search field loses focus, and "ret" in "return" is selected in the google.com source.
Attachments
Proposed Patch (2.09 KB, patch)
2009-04-03 17:18 PDT, Timothy Hatcher 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Geoffrey Garen
Comment 1 2009-02-16 16:13:48 PST
<rdar://problem/6592367>
Patrick Mueller
Comment 2 2009-03-20 07:48:17 PDT
Unable to reproduce in r41844 on Mac. Has this been fixed already?
Timothy Hatcher
Comment 3 2009-03-20 09:19:04 PDT
I can't reproduce either. Sounds like it might have been a temporary regression.
Timothy Hatcher
Comment 4 2009-03-20 09:21:13 PDT
Actually it still reproduces in the Scripts panel, and seems to only be a problem in that panel and Resources.
Timothy Hatcher
Comment 5 2009-03-20 09:22:14 PDT
Oddly focus looks like it is in the search field (there is still a focus ring) but there is no blinking insertion caret and no keyboard events reach the search field.
Timothy Hatcher
Comment 6 2009-04-03 17:18:08 PDT
Created attachment 29247 [details] Proposed Patch
Darin Adler
Comment 7 2009-04-03 17:26:09 PDT
Comment on attachment 29247 [details] Proposed Patch r=me
Timothy Hatcher
Comment 8 2009-04-03 17:42:01 PDT
Landed in r42221.
Note You need to log in before you can comment on or make changes to this bug.