Unifdef the old style system code.

RESOLVED FIXED in Firefox 61

Status

()

enhancement
RESOLVED FIXED
Last year
10 months ago

People

(Reporter: emilio, Assigned: emilio)

Tracking

(Depends on 1 bug)

unspecified
mozilla61
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox61 fixed)

Details

Attachments

(1 attachment)

PSA, this will break the MinGW build.
Flags: needinfo?(tom)
Flags: needinfo?(gk)
Depends on: 1447367
Depends on: 1447414
Filed Bug 1447414 to turn off MinGW.
Flags: needinfo?(tom)
Flags: needinfo?(gk)
Comment on attachment 8960666 [details]
Bug 1447358: Unifdef the old style system code. r?jwatt

Jonathan Watt [:jwatt] (needinfo? me) has approved the revision.

https://phabricator.services.mozilla.com/D779
Attachment #8960666 - Flags: review+
This is a good start, but for completeness I'll note that there's a whole lot of code that needs taken out related things like:

StyleBackendType::Gecko
IsStyledByServo
nsStyleContext::IsGecko
I guess this doesn't really depend on bug 1447414 since it's tier2. I want to land most of the cleanups today so that they don't conflict around (this patch is rather massive), so I don't think it's particularly worth to wait for that.
No longer depends on: 1447414
Pushed by ecoal95@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8e5206684eea
Unifdef the old style system code. r=jwatt
https://hg.mozilla.org/mozilla-central/rev/8e5206684eea
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Depends on: 1447781
Depends on: 1493580
You need to log in before you can comment on or make changes to this bug.