Closed
Bug 1416564
Opened 7 years ago
Closed 7 years ago
Try to remove the mozmm CSS unit
Categories
(Core :: CSS Parsing and Computation, enhancement, P3)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox59 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete, site-compat)
Attachments
(2 files)
It's used only on a handful of tests, and not in any spec. That sounds like an intent to unship to me.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → emilio
Comment hidden (mozreview-request) |
Keywords: dev-doc-needed,
site-compat
Comment hidden (mozreview-request) |
Assignee | ||
Comment 3•7 years ago
|
||
(Was going to ask dbaron for review too, but he's not currently accepting requests)
Updated•7 years ago
|
Priority: -- → P3
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8927663 [details]
Bug 1416564: Remove mozmm CSS unit.
https://reviewboard.mozilla.org/r/198940/#review204424
::: dom/interfaces/base/nsIDOMWindowUtils.idl:83
(Diff revision 2)
> * Cannot be accessed from unprivileged context (not content-accessible)
> */
> readonly attribute boolean docCharsetIsForced;
>
> /**
> + * Return a the conversion of a physical millimeter in CSS pixels.
s/a the/the/
Attachment #8927663 -
Flags: review?(cam) → review+
Pushed by ecoal95@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/ab20e3b3ef7b
Remove mozmm CSS unit. r=heycam
Comment 6•7 years ago
|
||
It looks like this reduced our build times on Windows:
== Change summary for alert #10551 (as of Tue, 14 Nov 2017 13:11:45 GMT) ==
Improvements:
8% build times summary windows2012-64-rusttests opt taskcluster-c4.4xlarge 2,561.31 -> 2,360.03
6% build times summary windows2012-32 opt taskcluster-c4.4xlarge 2,838.70 -> 2,666.70
5% build times summary windows2012-64 opt static-analysis taskcluster-c4.4xlarge2,784.12 -> 2,643.45
5% build times summary windows2012-32 opt static-analysis taskcluster-c4.4xlarge2,674.80 -> 2,544.93
For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=10551
Comment 7•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Assignee | ||
Comment 8•7 years ago
|
||
(In reply to Ionuț Goldan [:igoldan], Performance Sheriffing from comment #6)
> 8% build times summary windows2012-64-rusttests opt
> taskcluster-c4.4xlarge 2,561.31 -> 2,360.03
> 6% build times summary windows2012-32 opt taskcluster-c4.4xlarge
> 2,838.70 -> 2,666.70
> 5% build times summary windows2012-64 opt static-analysis
> taskcluster-c4.4xlarge2,784.12 -> 2,643.45
> 5% build times summary windows2012-32 opt static-analysis
> taskcluster-c4.4xlarge2,674.80 -> 2,544.93
I'm as fascinated by that as bug 1416741, but I guess the "-mozmm" special-case string check in style_derive every dimension was somehow slow on windows? shrug
Posted the site compatibility note: https://www.fxsitecompat.com/en-CA/docs/2017/mozmm-css-unit-is-no-longer-supported/
Comment 10•7 years ago
|
||
I have documented this on MDN, by:
* Adding a note to the Fx59 rel notes: https://developer.mozilla.org/en-US/Firefox/Releases/59#CSS_2
* Describing the change in appropriate places on https://developer.mozilla.org/en-US/docs/Web/CSS/length
Let me know if that looks OK. Thanks!
Keywords: dev-doc-needed → dev-doc-complete
Comment 11•7 years ago
|
||
Removing this valuable reason for preferring mozilla-based browsers to non-mozilla web browsers broke approximately 67 of my web pages using this stylesheet, and 172 of my web pages using the mozmm unit in their head and/or inline styles.
Many of these, maybe most, were constructed or reconstructed following the breakage of CSS version 3 by the removal of absolute units to take advantage of the unique workaround provided by the mozmm unit.
CSS specs provide no means AFAICT for any of these to be repaired.
You need to log in
before you can comment on or make changes to this bug.
Description
•