Closed
Bug 1386053
Opened 7 years ago
Closed 7 years ago
Enable DevTools tests in Stylo mode for Linux64
Categories
(Core :: CSS Parsing and Computation, enhancement)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: jryans, Assigned: jryans)
References
Details
(Whiteboard: [Stylo])
Attachments
(2 files)
To avoid regressions in DevTools behavior, let's enable e10s-on parallel only DevTools tests for Stylo on Linux64.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → jryans
Status: NEW → ASSIGNED
Comment 1•7 years ago
|
||
DevTools tests don't look quite ready to me... I'm going to enable mochitest except chrome, browser-chrome, a11y, devtools as the first stage of bug 1383992.
Assignee | ||
Comment 2•7 years ago
|
||
(In reply to Xidorn Quan [:xidorn] UTC+10 from comment #1) > DevTools tests don't look quite ready to me... Correct, there are still some failures remaining. I was planning to skip them for now, so we can at least make sure we keep the rest green.
Updated•7 years ago
|
Summary: Stylo: Enable DevTools tests in Stylo mode for Linux64 → Enable DevTools tests in Stylo mode for Linux64
Whiteboard: [Stylo]
Assignee | ||
Comment 3•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e83292331a0428380ba12803554f34c617d5d4d7
Assignee | ||
Comment 4•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=12684743f286c539669280d87e07e7a67ed27d50
Assignee | ||
Comment 5•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=d5f73135f4bba16d0450de2342236c1bbe6130a8
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 8•7 years ago
|
||
mozreview-review |
Comment on attachment 8893140 [details] Bug 1386053 - Skip remaining DevTools Stylo failures. https://reviewboard.mozilla.org/r/164136/#review169494
Attachment #8893140 -
Flags: review?(manishearth) → review+
Comment 9•7 years ago
|
||
mozreview-review |
Comment on attachment 8893141 [details] Bug 1386053 - Enable DevTools tests for Stylo Linux64. https://reviewboard.mozilla.org/r/164138/#review169496 ::: taskcluster/ci/test/test-sets.yml:261 (Diff revision 1) > - mochitest-clipboard > - mochitest-chrome > - xpcshell > + > +devtools-tests: > + - mochitest-devtools-chrome Maybe you can more this into `stylo-tests` directly?
Comment 10•7 years ago
|
||
mozreview-review |
Comment on attachment 8893141 [details] Bug 1386053 - Enable DevTools tests for Stylo Linux64. https://reviewboard.mozilla.org/r/164138/#review169804 Again, it would be good to have the data in seta before these are enabled but since they only one test suite hopefully the impact won't be as great as when the entire set of stylo suites were enabled.
Comment 11•7 years ago
|
||
mozreview-review |
Comment on attachment 8893141 [details] Bug 1386053 - Enable DevTools tests for Stylo Linux64. https://reviewboard.mozilla.org/r/164138/#review169806
Attachment #8893141 -
Flags: review?(kmoir) → review+
Assignee | ||
Comment 12•7 years ago
|
||
mozreview-review-reply |
Comment on attachment 8893141 [details] Bug 1386053 - Enable DevTools tests for Stylo Linux64. https://reviewboard.mozilla.org/r/164138/#review169496 > Maybe you can more this into `stylo-tests` directly? I've only been planning to run DevTools with Stylo on Linux64, whereas `stylo-tests` is being used on multiple platforms. I could still put this in `stylo-tests` and disable the other platforms via `run-on-projects`, but that feels a bit more cryptic to me.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 15•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=4714f5f0a31e96dbbb8d7e3a02330d79e8a3c021
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 18•7 years ago
|
||
Pushed by jryans@gmail.com: https://hg.mozilla.org/integration/autoland/rev/7d9e2aa06502 Skip remaining DevTools Stylo failures. r=manishearth https://hg.mozilla.org/integration/autoland/rev/a3f35ca41cce Enable DevTools tests for Stylo Linux64. r=kmoir
Comment 19•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7d9e2aa06502 https://hg.mozilla.org/mozilla-central/rev/a3f35ca41cce
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•