Closed
Bug 1528712
Opened 6 years ago
Closed 6 years ago
Remove some more dependencies from style in nsPresContext.
Categories
(Core :: CSS Parsing and Computation, enhancement, P3)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla67
| Tracking | Status | |
|---|---|---|
| firefox67 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
Just a few more left.
|
Assignee | |
Comment 1•6 years ago
|
||
|
|
Assignee | |
Comment 2•6 years ago
|
||
It's a global object, it doesn't have to be stored in nsFont. Pass it from the
caller like the user font set and co.
Depends on D20141
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d981515b874b
Remove a couple trivial dependencies on nsPresContext. r=heycam
https://hg.mozilla.org/integration/autoland/rev/2d5c4e71e258
Remove nsFont::featureValueLookup. r=jfkthame
|
||
Comment 4•6 years ago
|
||
Backed out 2 changesets (bug 1528712) for Linux spcshell and reftest failures (test_ext_browserSettings.js, 1022481-1.html)
push that caused the backout: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&selectedJob=229192047&revision=2d5c4e71e258e24f1a70ed850c619f97e56f2c60
backout: https://hg.mozilla.org/integration/autoland/rev/8a4ad06c05759fe087625f9fd8d047d848bee4c2
Flags: needinfo?(emilio)
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/autoland/rev/533f35feb9eb
Remove a couple trivial dependencies on nsPresContext. r=heycam
https://hg.mozilla.org/integration/autoland/rev/d15621c6f424
Remove nsFont::featureValueLookup. r=jfkthame
|
|
Assignee | |
Updated•6 years ago
|
Flags: needinfo?(emilio)
|
||
Comment 6•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/533f35feb9eb
https://hg.mozilla.org/mozilla-central/rev/d15621c6f424
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•