Remove support for -moz-calc()

RESOLVED FIXED in Firefox 53

Status

()

defect
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: bmo, Assigned: bmo)

Tracking

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

unspecified
mozilla53
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(5 attachments)

Assignee

Description

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

Updated

2 years 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)
Comment hidden (mozreview-request)
Blocks: unprefix
Summary: Remove the support of -moz-calc → Remove support for -moz-calc()
Assignee

Comment 12

2 years 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

2 years 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

2 years 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

2 years 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

2 years 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

2 years 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

2 years 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+
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Assignee

Comment 29

2 years ago
Try looks good. Ready to go.

Comment 30

2 years 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
Assignee

Comment 33

2 years 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.
Depends on: 1334064

Comment 35

2 years 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

2 years 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

2 years 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.