Closed
Bug 1357715
(stylo-devtools-tests)
Opened 4 years ago
Closed 4 years ago
[meta] Stylo: Pass DevTools tests
Categories
(Core :: CSS Parsing and Computation, enhancement, P5)
Core
CSS Parsing and Computation
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
Assignee | ||
Updated•4 years ago
|
Priority: -- → P5
Assignee | ||
Comment 1•4 years ago
|
||
Updated run now that a few crashing issues were fixed: https://treeherder.mozilla.org/#/jobs?repo=try&revision=5c5f4a1469f213ae25760cebfd5ea5f425b576ec
Assignee | ||
Comment 2•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f454b5577299a506ea812a53113368ad7ff991cc
Assignee | ||
Comment 3•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a4b1d029e64f2bc6712a7f156c09f62747cb38b8
Assignee | ||
Comment 4•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=355298c531857aab509eb9af074ec649a52c13a8
Assignee | ||
Comment 5•4 years ago
|
||
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.
Assignee | ||
Comment 6•4 years ago
|
||
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
Comment hidden (mozreview-request) |
Comment 8•4 years ago
|
||
mozreview-review |
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
Comment 10•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a7a5b2fc4f0a
Comment 11•4 years ago
|
||
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)
Comment 12•4 years ago
|
||
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
Assignee | ||
Comment 13•4 years ago
|
||
(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)
Comment hidden (mozreview-request) |
Comment 15•4 years ago
|
||
Pushed by jryans@gmail.com: https://hg.mozilla.org/integration/autoland/rev/e6dfe16f777b Skip DevTools tests on Stylo that use shadow DOM. r=heycam
Comment 16•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e6dfe16f777b
Assignee | ||
Comment 17•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=4d05347ab605c7ec8da028ce2dd3e56fec44529e
Assignee | ||
Comment 18•4 years ago
|
||
New failure count entry: 2017-04-19: 376 2017-05-23: 183 2017-06-07: 172 2017-06-16: 159 2017-06-29: 140
Assignee | ||
Comment 19•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2fdedc25e9243fb83ecd6b6dab0cb54af2559f6c
Assignee | ||
Comment 20•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=0fc2940d6f2d33a0d589de0b0947ec1e944e73e5
Assignee | ||
Comment 21•4 years ago
|
||
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
Assignee | ||
Comment 22•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9b31de0d8404ab0f925af3d00e831033391c3551
Assignee | ||
Comment 23•4 years ago
|
||
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
Assignee | ||
Comment 24•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=154e6fc2a776a80ced309d36c06465f7ea37e4b1
Assignee | ||
Comment 25•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=744fe0f4620dc84ff9cb57dae86152af6f8554e3
Assignee | ||
Comment 26•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=de0f92fdf53e94f72ef41312c8558a4820b62429
Assignee | ||
Comment 27•4 years ago
|
||
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
Updated•4 years ago
|
Blocks: stylo-mochitest
Assignee | ||
Comment 28•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e1995483b7fe673b23aa0fa8fc608910dfd8d1e2
Assignee | ||
Comment 29•4 years ago
|
||
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
Assignee | ||
Comment 30•4 years ago
|
||
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
Assignee | ||
Comment 31•4 years ago
|
||
All DevTools tests are active with Stylo!
Assignee: nobody → jryans
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox57:
--- → fixed
Keywords: leave-open
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•