GetCssTextImpl can probably go away

RESOLVED FIXED in Firefox 60

Status

()

enhancement
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Tracking

unspecified
mozilla60
Points:
---

Firefox Tracking Flags

(firefox60 fixed)

Details

Attachments

(1 attachment)

We used to have a const GetCssTextImpl and a non-const GetCssText.  But we should be able to just make GetCssText const everywhere and nix GetCssTextImpl.
Depends on: 1428610
Looks like we can't make nsICSSDeclaration::GetCssText const...
Gah, unfortunate :(
MozReview-Commit-ID: 5fejArxQckq
Attachment #8946773 - Flags: review?(emilio)
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Comment on attachment 8946773 [details] [diff] [review]
Mostly get rid of GetCssTextImpl

Review of attachment 8946773 [details] [diff] [review]:
-----------------------------------------------------------------

Beautiful, thanks!
Attachment #8946773 - Flags: review?(emilio) → review+

Comment 5

a year ago
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/765f21b51ad2
Mostly get rid of GetCssTextImpl.  r=emilio

Comment 6

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/765f21b51ad2
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.