In updateFocusCandidateIfCloser method (see WebCore/page/FocusController.cpp), we do all bail out checks in the begining of the method body. If after those bail out checks, no "best FocusCandidate" has been taken yet (i.e. focusCandidate.isNull() == true) , we can safely take the current candidate, and exit earlier.
no behavior change, it is just a safe assignment shortcut.
patch coming ...
Created attachment 53680 [details]
Comment on attachment 53680 [details]
Clearing flags on attachment: 53680
Committed r57921: <http://trac.webkit.org/changeset/57921>
Thank you Simon Fraser!
http://trac.webkit.org/changeset/57921 might have broken Leopard Intel Release (Tests)
The following changes are on the blame list:
Revision r57921 cherry-picked into qtwebkit-2.0 with commit 2be19c2e5f8f6902fb11932cc51a0a309eebd2d1