Remove support for -moz-calc()

RESOLVED FIXED in Firefox 53

Status

()

Core
CSS Parsing and Computation
RESOLVED FIXED
4 months ago
7 days ago

People

(Reporter: astley, Assigned: astley)

Tracking

(Blocks: 1 bug, {addon-compat, dev-doc-complete, site-compat})

unspecified
mozilla53
addon-compat, dev-doc-complete, site-compat
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox53 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(5 attachments)

(Assignee)

Description

4 months ago
This is a follow-up for bug 771678 comment 7.
I will remove prefixed CSS function -moz-calc from the tree.
(Assignee)

Updated

4 months ago
Assignee: nobody → aschen
Depends on: 771678
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 6

4 months ago
1st Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=25a81acc2aa13318fc15834de72f8e0ed33f10b0
Status: NEW → ASSIGNED
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Blocks: 775235
Summary: Remove the support of -moz-calc → Remove support for -moz-calc()

Updated

4 months ago
Keywords: addon-compat, dev-doc-needed, site-compat
(Assignee)

Comment 12

4 months ago
2nd Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=c552305ed4f78c03a599f814ab6c34bcb7e75249

Exclude changes in mask-image-2.html since it's just mistakenly included in first upload.
(Assignee)

Updated

4 months ago
Attachment #8827028 - Flags: review?(cam)
Attachment #8827029 - Flags: review?(cam)
Attachment #8827030 - Flags: review?(ttromey)
Attachment #8827031 - Flags: review?(cam)
Attachment #8827032 - Flags: review?(ttromey)

Comment 13

4 months ago
mozreview-review
Comment on attachment 8827028 [details]
Bug 1331296 : Part 1 - Remove or unprefix -moz-calc() from layout tests.

https://reviewboard.mozilla.org/r/104852/#review105826
Attachment #8827028 - Flags: review?(cam) → review+

Comment 14

4 months ago
mozreview-review
Comment on attachment 8827029 [details]
Bug 1331296 : Part 2 - Unprefix -moz-calc() in dom tests.

https://reviewboard.mozilla.org/r/104854/#review105828
Attachment #8827029 - Flags: review?(cam) → review+

Comment 15

4 months ago
mozreview-review
Comment on attachment 8827031 [details]
Bug 1331296 : Part 4 - Remove CSS parser support of -moz-calc().

https://reviewboard.mozilla.org/r/104858/#review105830

::: layout/style/nsCSSParser.cpp:7855
(Diff revision 2)
>  
>  static bool
>  IsCSSTokenCalcFunction(const nsCSSToken& aToken)
>  {
>    return aToken.mType == eCSSToken_Function &&
> -         (aToken.mIdent.LowerCaseEqualsLiteral("calc") ||
> +         (aToken.mIdent.LowerCaseEqualsLiteral("calc"));

Nit: remove the parens around the function call.
Attachment #8827031 - Flags: review?(cam) → review+
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 21

4 months ago
mozreview-review
Comment on attachment 8827030 [details]
Bug 1331296 : Part 3 - Remove -moz-calc() from DevTool tests.

https://reviewboard.mozilla.org/r/104856/#review105950

Thank you.
Attachment #8827030 - Flags: review?(ttromey) → review+

Comment 22

4 months ago
mozreview-review
Comment on attachment 8827032 [details]
Bug 1331296 : Part 5 - Update DevTool CSS property database.

https://reviewboard.mozilla.org/r/104860/#review105956

Thank you for doing this.  Looks good.
Attachment #8827032 - Flags: review?(ttromey) → review+
(Assignee)

Comment 23

4 months ago
3rd Try before landing: https://treeherder.mozilla.org/#/jobs?repo=try&revision=08771062c91c82f088fea96bdd54a08be40675cc
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 29

4 months ago
Try looks good. Ready to go.

Comment 30

4 months ago
Pushed by tlin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bcdb1c9f405b
Part 1 - Remove or unprefix -moz-calc() from layout tests. r=heycam
https://hg.mozilla.org/integration/autoland/rev/0cd431a10e07
Part 2 - Unprefix -moz-calc() in dom tests. r=heycam
https://hg.mozilla.org/integration/autoland/rev/b2a57de4472d
Part 3 - Remove -moz-calc() from DevTool tests. r=tromey
https://hg.mozilla.org/integration/autoland/rev/62fa067420ee
Part 4 - Remove CSS parser support of -moz-calc(). r=heycam
https://hg.mozilla.org/integration/autoland/rev/3ea611dc2979
Part 5 - Update DevTool CSS property database. r=tromey

Comment 31

4 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/bcdb1c9f405b
https://hg.mozilla.org/mozilla-central/rev/0cd431a10e07
https://hg.mozilla.org/mozilla-central/rev/b2a57de4472d
https://hg.mozilla.org/mozilla-central/rev/62fa067420ee
https://hg.mozilla.org/mozilla-central/rev/3ea611dc2979
Status: ASSIGNED → RESOLVED
Last Resolved: 4 months ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Posted the site compatibility doc: https://www.fxsitecompat.com/en-CA/docs/2017/moz-calc-has-been-removed/
(Assignee)

Comment 33

4 months ago
(In reply to Kohei Yoshino [:kohei] from comment #32)
> Posted the site compatibility doc:
> https://www.fxsitecompat.com/en-CA/docs/2017/moz-calc-has-been-removed/
Thank you, Kohei-san.
Added notes at the following pages:
https://developer.mozilla.org/en-US/docs/Web/CSS/calc#Browser_compatibility
https://developer.mozilla.org/en-US/Firefox/Releases/53#CSS

Sebastian
Keywords: dev-doc-needed → dev-doc-complete
Depends on: 1334064

Comment 35

7 days ago
Firefox has been having the unpleasant line-height bug, and -moz-calc() could help counteract the bug. Before Firefox's line-height issue gets fixed, making -moz-calc() unavailable is a bad approach in my opinion.
(Assignee)

Comment 36

7 days ago
(In reply to Ian Yang from comment #35)
> Firefox has been having the unpleasant line-height bug, and -moz-calc()
> could help counteract the bug. Before Firefox's line-height issue gets
> fixed, making -moz-calc() unavailable is a bad approach in my opinion.

Could you be more specific on the line-height bug that you mentioned here ?
It'd be great if you can provide a sameple or bug for reference.
Rather than using it as a workaround for Firefox line-height issue, I'd love to get it fixed sooner.
Flags: needinfo?(ian.html)

Comment 37

7 days ago
(In reply to Astley Chen [:astley] (UTC+8) from comment #36)
> (In reply to Ian Yang from comment #35)
> > Firefox has been having the unpleasant line-height bug, and -moz-calc()
> > could help counteract the bug. Before Firefox's line-height issue gets
> > fixed, making -moz-calc() unavailable is a bad approach in my opinion.
> 
> Could you be more specific on the line-height bug that you mentioned here ?
> It'd be great if you can provide a sameple or bug for reference.
> Rather than using it as a workaround for Firefox line-height issue, I'd love
> to get it fixed sooner.

Please see the following bug report for reference:
https://bugzilla.mozilla.org/show_bug.cgi?id=442139
Flags: needinfo?(ian.html)
You need to log in before you can comment on or make changes to this bug.