Bug 78026 - [Qt] REGRESSION(r106918): It made svg/as-object/nested-embedded-svg-size-changes.html fail in debug mode
Summary: [Qt] REGRESSION(r106918): It made svg/as-object/nested-embedded-svg-size-chan...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P1 Critical
Assignee: Nikolas Zimmermann
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 77736
  Show dependency treegraph
 
Reported: 2012-02-07 13:20 PST by Csaba Osztrogonác
Modified: 2012-02-09 06:05 PST (History)
7 users (show)

See Also:


Attachments
Patch (3.66 KB, patch)
2012-02-09 03:26 PST, Nikolas Zimmermann
no flags Details | Formatted Diff | Diff
Patch v2 (4.56 KB, patch)
2012-02-09 03:34 PST, Nikolas Zimmermann
kling: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2012-02-07 13:20:29 PST
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.
Comment 1 Csaba Osztrogonác 2012-02-07 14:03:15 PST
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.
Comment 2 Nikolas Zimmermann 2012-02-09 02:45:34 PST
Chromium with v8 sees a similar problem, see bug 77183.
Comment 3 Nikolas Zimmermann 2012-02-09 03:26:45 PST
*** Bug 77183 has been marked as a duplicate of this bug. ***
Comment 4 Nikolas Zimmermann 2012-02-09 03:26:46 PST
Created attachment 126270 [details]
Patch
Comment 5 Nikolas Zimmermann 2012-02-09 03:34:02 PST
Created attachment 126273 [details]
Patch v2
Comment 6 Nikolas Zimmermann 2012-02-09 03:44:26 PST
I'll land this now, and see whether Qt stays green, and watch the cr flakiness dashboard, for any changes.
Comment 7 Nikolas Zimmermann 2012-02-09 03:45:26 PST
Committed r107213: <http://trac.webkit.org/changeset/107213>
Comment 8 Nikolas Zimmermann 2012-02-09 04:26:21 PST
(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.
Comment 9 Nikolas Zimmermann 2012-02-09 05:00:26 PST
Reopening, not convinced its fixed yet.
Comment 10 Nikolas Zimmermann 2012-02-09 06:05:47 PST
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.