RESOLVED FIXED Bug 100231
CSS3 Media Queries - add resolution tests to W3C test suite
https://bugs.webkit.org/show_bug.cgi?id=100231
Summary CSS3 Media Queries - add resolution tests to W3C test suite
Alexander Shalamov
Reported 2012-10-24 04:38:49 PDT
Resolution tests were removed, since implementation was missing. Since resolution feature was added recently, tests could be added back.
Attachments
Patch 1 (4.77 KB, patch)
2012-10-24 04:48 PDT, Alexander Shalamov
kenneth: review+
webkit.review.bot: commit-queue-
Patch 2 (93.20 KB, patch)
2012-10-24 07:03 PDT, Alexander Shalamov
no flags
Alexander Shalamov
Comment 1 2012-10-24 04:48:22 PDT
Created attachment 170365 [details] Patch 1 - Added resolution media feature tests to W3C Media Queries test suite
Kenneth Rohde Christiansen
Comment 2 2012-10-24 04:59:13 PDT
Comment on attachment 170365 [details] Patch 1 Not all platforms will pass with this and they will need different results. How do you intent to rebase that? After this lands?
WebKit Review Bot
Comment 3 2012-10-24 05:36:53 PDT
Comment on attachment 170365 [details] Patch 1 Attachment 170365 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/14566184 New failing tests: fast/media/w3c/test_media_queries.html
Alexander Shalamov
Comment 4 2012-10-24 07:03:42 PDT
Created attachment 170389 [details] Patch 2 - Added platform specific test expectations to blackberry, chromium, mac, win and gtk ports - Added assert_false and assert_true wrappers, so that stack trace is not shown when the test is failed
WebKit Review Bot
Comment 5 2012-10-24 07:38:24 PDT
Comment on attachment 170389 [details] Patch 2 Clearing flags on attachment: 170389 Committed r132352: <http://trac.webkit.org/changeset/132352>
WebKit Review Bot
Comment 6 2012-10-24 07:38:29 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.