Closed
Bug 1364845
Opened 7 years ago
Closed 7 years ago
stylo: add support for a "re-run selector matching on children only" restyle hint
Categories
(Core :: CSS Parsing and Computation, enhancement)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: heycam, Assigned: heycam)
References
Details
Attachments
(1 file)
This is something that Gecko doesn't do (though could), but I had written down in my notes, and wasn't too hard.
Assignee | ||
Comment 1•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=d7ddf9a3df019bfa707b8b6804bfdce3f2a025fd
Assignee | ||
Comment 2•7 years ago
|
||
https://github.com/servo/servo/pull/16906
Comment hidden (mozreview-request) |
Assignee | ||
Comment 4•7 years ago
|
||
Actually I'm not sure if these adjustments are needed due to this bug, or bug 1289868, since I've been testing them together. Filed bug 1366206 for the MathML one. 399384-1.html is a ::first-line test, so don't put too much stock in it passing now. For the test_transitions_and_reframes.html failures, it's this: TEST-UNEXPECTED-FAIL | layout/style/test/test_transitions_and_reframes.html | margin of child of element b2 (classes: t m o) ::before pseudo-element - got "100px", expected "25px" TEST-UNEXPECTED-FAIL | layout/style/test/test_transitions_and_reframes.html | margin of child of element a2 (classes: t m o) ::after pseudo-element - got "100px", expected "25px" TEST-UNEXPECTED-FAIL | layout/style/test/test_transitions_and_reframes.html | margin of child of element b2 (classes: m o t) ::before pseudo-element - got "100px", expected "25px" TEST-UNEXPECTED-FAIL | layout/style/test/test_transitions_and_reframes.html | margin of child of element a2 (classes: m o t) ::after pseudo-element - got "100px", expected "25px" Given the current implementation state of transitions and pseudo-elements, do you think it's not unexpected that this could start failing? If you think I should look into this, I can, though I'd rather land these patches first since it's taken me a while to get them to the point of being landable and don't want to do any more rebasing. :-)
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8869371 [details] Bug 1364845 - stylo: Adjust test expectations. https://reviewboard.mozilla.org/r/141018/#review144592 I think the transitions in pseudo-element tests are worth looking into, since when I was writing bug 1331047 the failures revealed some legit bugs. That being said, I think it's fine to investigate them as a followup, so r=me with a followup bug referenced from the expectations file.
Attachment #8869371 -
Flags: review?(emilio+bugs) → review+
Comment hidden (mozreview-request) |
Assignee | ||
Comment 7•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=73e544fdd4693b0f7b54b3a9d9392b278f38bee3
Pushed by canaltinova@gmail.com: https://hg.mozilla.org/integration/autoland/rev/36678242b5a6 stylo: Adjust test expectations. r=emilio
Comment 9•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/36678242b5a6
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → cam
You need to log in
before you can comment on or make changes to this bug.
Description
•