1386053 - Enable DevTools tests in Stylo mode for Linux64

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement)

Description

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

To avoid regressions in DevTools behavior, let's enable e10s-on parallel only DevTools tests for Stylo on Linux64.

Comment 1

[Xidorn Quan [:xidorn] UTC+10]

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.

Comment 2

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

(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.

Comment 3

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e83292331a0428380ba12803554f34c617d5d4d7

Comment 4

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=12684743f286c539669280d87e07e7a67ed27d50

Comment 5

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d5f73135f4bba16d0450de2342236c1bbe6130a8

Comment 6

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Attachment: Bug 1386053 - Skip remaining DevTools Stylo failures.

Review commit: https://reviewboard.mozilla.org/r/164136/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/164136/

Comment 7

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Attachment: Bug 1386053 - Enable DevTools tests for Stylo Linux64.

Review commit: https://reviewboard.mozilla.org/r/164138/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/164138/

Comment 8

[Manish Goregaokar [:manishearth]]

Comment on attachment 8893140 [details]
Bug 1386053 - Skip remaining DevTools Stylo failures.

https://reviewboard.mozilla.org/r/164136/#review169494

Comment 9

[Xidorn Quan [:xidorn] UTC+10]

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

[Kim Moir [:kmoir] ET]

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

[Kim Moir [:kmoir] ET]

Comment on attachment 8893141 [details]
Bug 1386053 - Enable DevTools tests for Stylo Linux64.

https://reviewboard.mozilla.org/r/164138/#review169806

Comment 12

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

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 13

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8893140 [details]
Bug 1386053 - Skip remaining DevTools Stylo failures.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/164136/diff/1-2/

Comment 14

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8893141 [details]
Bug 1386053 - Enable DevTools tests for Stylo Linux64.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/164138/diff/1-2/

Comment 15

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4714f5f0a31e96dbbb8d7e3a02330d79e8a3c021

Comment 16

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8893140 [details]
Bug 1386053 - Skip remaining DevTools Stylo failures.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/164136/diff/2-3/

Comment 17

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8893141 [details]
Bug 1386053 - Enable DevTools tests for Stylo Linux64.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/164138/diff/2-3/

Comment 18

[Pulsebot]

Pushed by [e-mail]:
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

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/7d9e2aa06502
https://hg.mozilla.org/mozilla-central/rev/a3f35ca41cce