After r106918 svg/as-object/nested-embedded-svg-size-changes.html started to fail in debug mode: --- /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/layout-test-results/svg/as-object/nested-embedded-svg-size-changes-expected.txt +++ /home/webkitbuildbot/slaves/debug64bit/buildslave/qt-linux-64-debug/build/layout-test-results/svg/as-object/nested-embedded-svg-size-changes-actual.txt @@ -1,3 +1,5 @@ +CONSOLE MESSAGE: line 9: TypeError: 'undefined' is not an object (evaluating 'svgDoc.defaultView') +FAIL: Timed out waiting for notifyDone to be called This test checks that intrinsic size changes of an embedded SVG cause immediate updates of the object that embeds it. @@ -12,12 +14,3 @@ PASS document.defaultView.getComputedStyle(object1).width is "400px" PASS document.defaultView.getComputedStyle(object1).height is "200px" -Check final <object> size, after resizing finished -PASS document.defaultView.getComputedStyle(object1).width is "400px" -PASS document.defaultView.getComputedStyle(object1).height is "200px" - -Test passed if you see two green rectangles -PASS successfullyParsed is true - -TEST COMPLETE - svg/as-object/nested-embedded-svg-size-changes.html passes with singly run, but you can reproduce this fail easily with run svg/as-object tests in debug mode.
I skipped this test to make buildbot green to be able catch new regressions - http://trac.webkit.org/changeset/106989 Please unskip it with the proper fix.
Chromium with v8 sees a similar problem, see bug 77183.
*** Bug 77183 has been marked as a duplicate of this bug. ***
Created attachment 126270 [details] Patch
Created attachment 126273 [details] Patch v2
I'll land this now, and see whether Qt stays green, and watch the cr flakiness dashboard, for any changes.
Committed r107213: <http://trac.webkit.org/changeset/107213>
(In reply to comment #7) > Committed r107213: <http://trac.webkit.org/changeset/107213> Oops, had a last minute typo, fixed in r107218 - now I hope it works.
Reopening, not convinced its fixed yet.
The fix from r107213 made it work on Qt Debug, according to: http://build.webkit.sed.hu/builders/x86-64%20Linux%20Qt%20Debug/builds/20988. Closing bug.