CLOSED FIXED Bug 41157
Spatial Navigation: refactor spatial-navigation-utils.js to support testing nested frames deeper than one level 
https://bugs.webkit.org/show_bug.cgi?id=41157
Summary Spatial Navigation: refactor spatial-navigation-utils.js to support testing n...
Antonio Gomes
Reported 2010-06-24 07:12:07 PDT
Currently, it is not possible to test focused elements in {i}frames nested in more than one level deep (see sample below): <html> <body> <iframe width="50%" height="100px" frameborder="1" src="resources/iframe.html"></iframe><br> <body> <iframe width='80%' height="70px" frameborder="0" src="data:text/html, <body> <a id='1' href='#'>a</a> patch coming ...
Attachments
(commit r61819, r=kenneth) patch v1 (28.39 KB, patch)
2010-06-24 07:29 PDT, Antonio Gomes 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Antonio Gomes
Comment 1 2010-06-24 07:29:58 PDT
Created attachment 59653 [details] (commit r61819, r=kenneth) patch v1 The output of all spatial navigation changed , but these are string changes only, so this part of the patch can be "ignored". e.g. -PASS document.activeElement.getAttribute("id") is "end" +PASS gFocusedDocument.activeElement.getAttribute("id") is "end"
Antonio Gomes
Comment 2 2010-06-24 22:16:52 PDT
Comment on attachment 59653 [details] (commit r61819, r=kenneth) patch v1 Clearing flags on attachment: 59653 Committed r61819: <http://trac.webkit.org/changeset/61819>
Antonio Gomes
Comment 3 2010-07-01 06:15:25 PDT
@tronical, cherry-picking this one, would make Spatial navigation layouttests in QtWebKit-2.0 branch green.
Simon Hausmann
Comment 4 2010-07-01 12:54:07 PDT
Revision r61819 cherry-picked into qtwebkit-2.0 with commit fe2f92a1322131a92d23273b84fe672257b46388
Note You need to log in before you can comment on or make changes to this bug.