The default bug view has changed. See this FAQ.

Rip out cairo+pixman from libxul

RESOLVED FIXED in mozilla15

Status

()

Core
Build Config
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
mozilla15
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 3 obsolete attachments)

(Assignee)

Description

5 years ago
+++ This bug was initially created as a clone of Bug #750747 +++

cairo+pixman needs to go outside of libxul, too!
(Assignee)

Comment 1

5 years ago
Created attachment 620396 [details] [diff] [review]
Part 1: Take out the debugging code which we no longer use
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #620396 - Flags: review?(jmuizelaar)
Attachment #620396 - Flags: review?(jmuizelaar) → review+
(Assignee)

Comment 2

5 years ago
Created attachment 620431 [details] [diff] [review]
Part 2: Split out cairo+pixman into gkmedias.dll
Attachment #620431 - Flags: review?(khuey)
Attachment #620431 - Flags: review?(khuey) → review+
(Assignee)

Comment 3

5 years ago
Landed the first part: https://hg.mozilla.org/integration/mozilla-inbound/rev/a642269f01a2
(Assignee)

Comment 4

5 years ago
And the second part: https://hg.mozilla.org/integration/mozilla-inbound/rev/27c51f14c3ed
Target Milestone: --- → mozilla15
(Assignee)

Comment 5

5 years ago
Backed out the last part because of build bustage: https://hg.mozilla.org/integration/mozilla-inbound/rev/070534825e25
(Assignee)

Comment 6

5 years ago
Created attachment 620464 [details] [diff] [review]
Part 2: Split out cairo+pixman into gkmedias.dll

This version of the patch actually does compile!
Attachment #620431 - Attachment is obsolete: true
Attachment #620464 - Flags: review?(ted.mielczarek)
Attachment #620464 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Comment 7

5 years ago
Created attachment 620471 [details] [diff] [review]
Part 2: Split out cairo+pixman into gkmedias.dll

This should work on Linux as well, I'll request review when my build finishes.
Attachment #620464 - Attachment is obsolete: true
(Assignee)

Comment 8

5 years ago
Created attachment 620482 [details] [diff] [review]
Part 2: Split out cairo+pixman into gkmedias.dll
Attachment #620471 - Attachment is obsolete: true
Attachment #620482 - Flags: review?(khuey)
Attachment #620482 - Flags: review?(khuey) → review+
(Assignee)

Comment 9

5 years ago
Part 2: https://hg.mozilla.org/integration/mozilla-inbound/rev/828281d69978
(Assignee)

Comment 10

5 years ago
https://hg.mozilla.org/mozilla-central/rev/a642269f01a2
https://hg.mozilla.org/mozilla-central/rev/27c51f14c3ed
https://hg.mozilla.org/mozilla-central/rev/070534825e25
https://hg.mozilla.org/mozilla-central/rev/828281d69978
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Depends on: 751521
You need to log in before you can comment on or make changes to this bug.