[Webkit-unassigned] [Bug 36923] [Chromium]: FindInPage issue with focus remaining on element after match is found

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=36923





--- Comment #4 from Ojan Vafai <ojan at chromium.org>  2010-04-01 08:16:31 PST ---
(From update of attachment 52245)
Code looks correct to me.

> Index: WebKit/chromium/ChangeLog
> ===================================================================
> --- WebKit/chromium/ChangeLog	(revision 56886)
> +++ WebKit/chromium/ChangeLog	(working copy)
> @@ -1,3 +1,15 @@
> +2010-03-31  Finnur Thorarinsson  <finnur.webkit at gmail.com>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        [chromium] FindInPage should clear the focused node when a match has
> +        been found.
> +
> +        https://bugs.webkit.org/show_bug.cgi?id=36923


Would be nice to have a description here about how this fixes the bug, i.e.
that WebFrameImpl::setFocus will try to refocus editable elements if it thinks
they have focus, causing the page to scroll.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.