Closed
Bug 1291657
Opened 8 years ago
Closed 8 years ago
run web-platform-tests for linux64 in ubuntu 16.04 vs ubuntu 12.04
Categories
(Testing :: web-platform-tests, defect)
Tracking
(firefox51 fixed)
RESOLVED
FIXED
mozilla51
Tracking | Status | |
---|---|---|
firefox51 | --- | fixed |
People
(Reporter: jmaher, Unassigned)
References
Details
Attachments
(2 files)
Bug 1291657 - run web-platform-tests for linux64 in ubuntu 16.04 vs ubuntu 12.04 [manifest changes].
(deleted),
text/x-review-board-request
|
jgraham
:
review+
|
Details |
(deleted),
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/68900/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/68900/
Attachment #8777312 -
Flags: review?(james)
Updated•8 years ago
|
Attachment #8777312 -
Flags: review?(james) → review+
Comment 2•8 years ago
|
||
Comment on attachment 8777312 [details]
Bug 1291657 - run web-platform-tests for linux64 in ubuntu 16.04 vs ubuntu 12.04 [manifest changes].
https://reviewboard.mozilla.org/r/68900/#review66716
::: testing/web-platform/meta/content-security-policy/object-src/object-src-2_1.html.ini:7
(Diff revision 1)
> type: testharness
> [Async SWF load test]
> - expected: NOTRUN
> + expected:
> + if (os=="linux") and (version=="Ubuntu 16.04"): PASS
> + if (os=="linux") and (version=="Ubuntu 12.04"): NOTRUN
> + if not (os=="linux"): NOTRUN
Note that this will change the default for other versions of Linux to `PASS` (compared to `NOTRUN` before). It isn't clear if this is better or worse really.
::: testing/web-platform/meta/referrer-policy/no-referrer-when-downgrade/attr-referrer/cross-origin/http-http/img-tag/insecure-protocol.keep-origin-redirect.http.html.ini:4
(Diff revision 1)
> [insecure-protocol.keep-origin-redirect.http.html]
> type: testharness
> - expected: TIMEOUT
> + expected:
> + if (os=="linux") and (version=="Ubuntu 16.04"): OK
I find it very odd that these tests suddenly work in 16.04 but not on any other OS. Do we understand that?
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7f3080922905
run web-platform-tests for linux64 in ubuntu 16.04 vs ubuntu 12.04 [manifest changes]. r=jgraham
Comment 4•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Reporter | ||
Comment 5•8 years ago
|
||
that was just the manifest changes, what remains is a small patch to run the tests on 16.04 :)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reporter | ||
Comment 6•8 years ago
|
||
Attachment #8781136 -
Flags: review?(dustin)
Comment 8•8 years ago
|
||
Comment on attachment 8781136 [details] [diff] [review]
switch wpt tests to use desktop1604-test
Review of attachment 8781136 [details] [diff] [review]:
-----------------------------------------------------------------
This is the same patch I wrote in bug 1294538. On a technical level it is correct, so I'm giving r+.
Attachment #8781136 -
Flags: review?(dustin) → review+
Reporter | ||
Comment 9•8 years ago
|
||
thanks!
Comment 10•8 years ago
|
||
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b97908a2c4b5
run web-platform-tests for linux64 in ubuntu 16.04 vs ubuntu 12.04. r=gps
Comment 11•8 years ago
|
||
bugherder |
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•