Closed Bug 1357715 (stylo-devtools-tests) Opened 3 years ago Closed 2 years ago

[meta] Stylo: Pass DevTools tests

Categories

(Core :: CSS Parsing and Computation, enhancement, P5)

enhancement

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: jryans, Assigned: jryans)

References

(Blocks 1 open bug)

Details

(Keywords: meta)

Attachments

(1 file)

We should start to look at DevTools test failures with Stylo enabled since DevTools uses additional code paths that might not be hit when just styling content normally.

An example try run[1] shows the current state.

[1]: https://treeherder.mozilla.org/#/jobs?repo=try&revision=752bed7671c8df3c830b37df3115f5aa647c6428
Priority: -- → P5
Depends on: 1357716
Depends on: 1357724
Depends on: 1357732
Depends on: 1367615
Depends on: 1367620
Depends on: 1368749
Depends on: 1371388
Depends on: 1371393
Depends on: 1371395
Depends on: 1371410
Depends on: 1371457
Depends on: 1371478
Depends on: 1371708
Some historical DevTools failure counts (looking at e10s opt only):

2017-04-19: 376
2017-05-23: 183
2017-06-07: 172
2017-06-16: 159

So, making progress at least!  Some runs do crash though, so potentially more failures will be revealed once crashes are fixed.
Depends on: 1375150
Marking leave-open to use this bug to land some DevTools expectation updates for platform features that don't block shipping Stylo.
Keywords: leave-open
Depends on: 1375230
Comment on attachment 8880071 [details]
Bug 1357715 - Skip DevTools tests on Stylo that use shadow DOM.

https://reviewboard.mozilla.org/r/151392/#review156538
Attachment #8880071 - Flags: review?(cam) → review+
Pushed by jryans@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/a7a5b2fc4f0a
Skip DevTools tests on Stylo that use shadow DOM. r=heycam
this caused perma failures after this landed on m-c like https://treeherder.mozilla.org/logviewer.html#?job_id=109517923&repo=mozilla-central
Flags: needinfo?(jryans)
Backout by cbook@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/594cc32b6323
Backed out changeset a7a5b2fc4f0a for causing perma failures after merged to m-c
(In reply to Carsten Book [:Tomcat] from comment #11)
> this caused perma failures after this landed on m-c like
> https://treeherder.mozilla.org/logviewer.html#?job_id=109517923&repo=mozilla-
> central

Bug 1375699 made the pref optional again, so we need to keep the try blocks.
Flags: needinfo?(jryans)
Depends on: 1375952
Pushed by jryans@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/e6dfe16f777b
Skip DevTools tests on Stylo that use shadow DOM. r=heycam
New failure count entry:

2017-04-19: 376
2017-05-23: 183
2017-06-07: 172
2017-06-16: 159
2017-06-29: 140
Depends on: 1374881
New failure count entry:

2017-04-19: 376
2017-05-23: 183
2017-06-07: 172
2017-06-16: 159
2017-06-29: 140
2017-07-11: 146
Depends on: 1379985
No longer depends on: 1374881
Depends on: 1380890
Depends on: 1381083
Depends on: 1381137
Depends on: 1381143
New failure count entry:

2017-04-19: 376
2017-05-23: 183
2017-06-07: 172
2017-06-16: 159
2017-06-29: 140
2017-07-11: 146
2017-07-12: 180
New failure count entry:

2017-04-19: 376
2017-05-23: 183
2017-06-07: 172
2017-06-16: 159
2017-06-29: 140
2017-07-11: 146
2017-07-12: 180
2017-07-24: 171
New failure count entry:

2017-04-19: 376
2017-05-23: 183
2017-06-07: 172
2017-06-16: 159
2017-06-29: 140
2017-07-11: 146
2017-07-12: 180
2017-07-24: 171
2017-07-25: 53
Depends on: 1384789
Depends on: 1384014
Depends on: 1384802
Blocks: 1386053
New failure count entry:

2017-04-19: 376
2017-05-23: 183
2017-06-07: 172
2017-06-16: 159
2017-06-29: 140
2017-07-11: 146
2017-07-12: 180
2017-07-24: 171
2017-07-25: 53
2017-08-01: 46
Depends on: 1386384
Depends on: 1386862
Depends on: 1386865
Depends on: 1387445
Depends on: 1387905
Depends on: 1387906
Depends on: 1388165
Depends on: 1391198
Depends on: 1394994
All DevTools tests are active with Stylo!
Assignee: nobody → jryans
Status: NEW → RESOLVED
Closed: 2 years ago
Keywords: leave-open
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Keywords: meta
You need to log in before you can comment on or make changes to this bug.