Closed
Bug 1346047
Opened 8 years ago
Closed 8 years ago
stylo: Enable chrome mochitests of style system and enable mochitests on e10s
Categories
(Core :: CSS Parsing and Computation, enhancement, P1)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: xidorn, Assigned: xidorn)
References
Details
Attachments
(1 file)
There are two browser mochitests and nine chrome mochitests in layout/style/test. We probably want to enable them as well since they are also testing style system.
Updated•8 years ago
|
Priority: -- → P1
Assignee | ||
Updated•8 years ago
|
Summary: stylo: Enable browser and chrome mochitests of style system → stylo: Enable browser and chrome mochitests of style system and enable mochitests on e10s
Assignee | ||
Comment 2•8 years ago
|
||
It looks like browser-chrome mochitest doesn't use SimpleTest framework, so integrating failure pattern into them is probably non-trivial. Given that there are only two browser-chrome mochitests, and only one of them is actually testing something new, I'm going to leave it untouched for now.
Summary: stylo: Enable browser and chrome mochitests of style system and enable mochitests on e10s → stylo: Enable chrome mochitests of style system and enable mochitests on e10s
Assignee | ||
Comment 3•8 years ago
|
||
Comment hidden (mozreview-request) |
Assignee | ||
Updated•8 years ago
|
Attachment #8845750 -
Flags: review?(glob) → review?(jmaher)
Comment 5•8 years ago
|
||
mozreview-review |
Comment on attachment 8845750 [details]
Bug 1346047 - Enable chrome style system mochitests and run style system mochitests on e10s.
https://reviewboard.mozilla.org/r/118888/#review120960
thanks, this patch is looking good.
Attachment #8845750 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 6•8 years ago
|
||
Thanks for reviewing.
Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d8e496463611
Enable chrome style system mochitests and run style system mochitests on e10s. r=jmaher
Comment 8•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•