RESOLVED FIXED Bug 90594
REGRESSION(r121891): It made two tests fail on Qt and on Lion
https://bugs.webkit.org/show_bug.cgi?id=90594
Summary REGRESSION(r121891): It made two tests fail on Qt and on Lion
Kristóf Kosztyó
Reported 2012-07-05 02:18:34 PDT
After the r121891 these two tests start to fail. http/tests/inspector/console-cd-completions.html: --- /ramdisk/qt-linux-64-release/build/layout-test-results/http/tests/inspector/console-cd-completions-expected.txt +++ /ramdisk/qt-linux-64-release/build/layout-test-results/http/tests/inspector/console-cd-completions-actual.txt @@ -1,6 +1,6 @@ Test that completions in the context of an iframe with a different origin will result in names of its global variables. Test passes if all global variables are found among completions AND there are NO console messages. Bug 65457. -myGlobalVar -myGlobalFunction +NOT FOUND: myGlobalVar +NOT FOUND: myGlobalFunction http/tests/inspector/console-cd.html: --- /ramdisk/qt-linux-64-release/build/layout-test-results/http/tests/inspector/console-cd-expected.txt +++ /ramdisk/qt-linux-64-release/build/layout-test-results/http/tests/inspector/console-cd-actual.txt @@ -1,5 +1,5 @@ Tests that console evaluation can be performed in an iframe context.Bug 19872. -foo = 2011 +foo = ReferenceError: Can't find variable: foo
Attachments
Csaba Osztrogonác
Comment 1 2012-07-05 02:48:12 PDT
No, it isn't Qt specific bug, it fails on Apple Lion too. (http://build.webkit.org/builders/Apple%20Lion%20Release%20WK1%20%28Tests%29/builds/844 - It isn't finished, but you can see these failing tests in the stdio) It's strange for me that layout tests takes more than an hour on the Apple bots with 8 paralel threads. It takes for Qt port 18 minutes on 1 thread.
Csaba Osztrogonác
Comment 2 2012-07-05 03:52:00 PDT
It is skipped on Qt by https://trac.webkit.org/changeset/121894 . Please unskip it with the proper fix. Thanks.
Peter Wang
Comment 3 2012-07-05 18:28:17 PDT
It's weird. I can manually do the test well. I need to investigate the DRT test.
Csaba Osztrogonác
Comment 4 2012-10-09 01:08:00 PDT
They pass on Qt now, I unskipped them - r130733
Note You need to log in before you can comment on or make changes to this bug.