[Webkit-unassigned] [Bug 40029] Widgets get 2 notifications (lost focus/got focus) each time focus switches between 2 nodes

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Stuart Morgan <stuartmorgan at chromium.org>  2010-06-02 10:37:54 PST ---
(drive-by, since I'm not actually a reviewer)

Isn't this introducing ordering problems? There are a lot of focus-related calls that happen in the main |if (newFocusedNode)| block, which used to happen after the setFocus(false) (which seems logical), but would now happen before it (which doesn't).

Granted the setFocus(false) call is new, and this may well not break anything that currently exists, but it seems like it could lead to weird and hard-to-find problems later if people do start using it.

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