Summary: | unskip webarchive tests on mac | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Alex Christensen <achristensen> | ||||||||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | benjamin, commit-queue, thorton | ||||||||||||
Priority: | P3 | ||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | Mac | ||||||||||||||
OS: | OS X 10.8 | ||||||||||||||
Attachments: |
|
Description
Alex Christensen
2013-05-08 12:27:09 PDT
Created attachment 201097 [details]
Patch
Comment on attachment 201097 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=201097&action=review > LayoutTests/platform/mac-wk2/TestExpectations:328 > +### START OF (5) Features that are disabled elsewhere re-enabled here add an "and" after elsewhere. > LayoutTests/platform/mac-wk2/TestExpectations:334 > +# dumpDOMAsWebArchive only supported for mac "is", "Mac", and perhaps this should be "Override global WebKit2 skip of dumpDOMAsWebArchive because it's implemented for Mac" or something. > LayoutTests/platform/wk2/TestExpectations:97 > +# dumpDOMAsWebArchive only implemented for mac add an "is" before only and capitalize Mac. Also, should these be in the global TestExpectations? Are they also skipped on WK1 for non-Mac platforms? Created attachment 201102 [details]
Patch
Comment on attachment 201102 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=201102&action=review > LayoutTests/ChangeLog:12 > + Replaced list of webarchive tests with webarchive test directories. > + * platform/wk2/TestExpectations: > + Unskip webarchive tests for Mac, note which webarchive tests still fail with bug numbers. I think these comments are under the wrong filenames (swap them?) > LayoutTests/platform/mac-wk2/TestExpectations:371 > +# setIconDatabaseEnabled not implemented in WebKitTestRunner > +# https://bugs.webkit.org/show_bug.cgi?id=115809 > +webarchive/test-link-rel-icon-beforeload.html [ Skip ] > + > +# https://bugs.webkit.org/show_bug.cgi?id=82665 > +http/tests/webarchive/test-css-url-encoding.html [ Skip ] > +http/tests/webarchive/test-css-url-encoding-shift-jis.html [ Skip ] > +http/tests/webarchive/test-css-url-encoding-utf-8.html [ Skip ] > +webarchive/test-css-url-resources-in-stylesheets.html [ Skip ] > +webarchive/test-css-url-resources-inline-styles.html [ Skip ] Does it make sense for these to be in this section? I'll do the swap, but I think it does make sense to put the skipped webarchive tests into the section called "Features that are disabled elsewhere and re-enabled here" because when they are fixed, they will only be enabled for Mac WK2 Created attachment 201103 [details]
Patch
How much would would it be to just support that on WebKit1? (In reply to comment #7) > How much would would it be to just support that on WebKit1? webarchives are already supported on Mac WebKit and I just added support for testing them with WK2, but they're a Mac-only feature that use CoreFoundation Comment on attachment 201103 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=201103&action=review > LayoutTests/platform/wk2/TestExpectations:99 > +# non-Mac ports don't support webarchives > +webarchive > +svg/webarchive I would prefer if you add those skip in EFL/QT/GTK's TestExpectations. Otherwise, it would be really easy to add a test skipped on WK2 by accident. Created attachment 201296 [details]
Patch
Comment on attachment 201296 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=201296&action=review > LayoutTests/platform/efl-wk2/TestExpectations:289 > +# non-Mac ports don't support webarchives Webkit comment style: comments are complete sentences with Upper case for the first character and a period. non-Mac port don't -> EFL doesn't. > LayoutTests/platform/gtk-wk2/TestExpectations:199 > +# non-Mac ports don't support webarchives ditto. > LayoutTests/platform/qt-5.0-mac-wk2/TestExpectations:86 > +# QT Mac port doesn't support webarchives QT -> Qt. > LayoutTests/platform/qt-5.0-wk2/TestExpectations:104 > +# non-Mac ports don't support webarchives ditto. > LayoutTests/platform/wk2/TestExpectations:97 > +# setIconDatabaseEnabled not implemented in WebKitTestRunner Comment style. Created attachment 201305 [details]
Patch
Comment on attachment 201305 [details] Patch Clearing flags on attachment: 201305 Committed r149859: <http://trac.webkit.org/changeset/149859> All reviewed patches have been landed. Closing bug. |