Closed
Bug 960524
Opened 12 years ago
Closed 11 years ago
Remove Image::DeprecatedGetAsSurface
Categories
(Core :: Graphics: Layers, defect)
Tracking
()
RESOLVED
FIXED
mozilla31
Tracking | Status | |
---|---|---|
firefox31 | --- | fixed |
People
(Reporter: ali, Assigned: jwatt)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
33.83 KB,
patch
|
mattwoodrow
:
review+
|
Details | Diff | Splinter Review |
Remove calls to Image::DeprecatedGetAsSurface and use Image::GetAsSourceSurface instead.
Comment 1•12 years ago
|
||
Marking this as NEW based on the conversation in bug 947194.
Status: UNCONFIRMED → NEW
Ever confirmed: true
![]() |
Assignee | |
Comment 2•11 years ago
|
||
The work in the various blocking bugs needs to be finished and then this should do the rest.
![]() |
Assignee | |
Comment 3•11 years ago
|
||
To land this the DeprecatedGetCurrentAsSurface() call in BasicImageLayer::GetAsSurface() needs to go away, which Matt is working on in bug 989858.
Depends on: 989858
![]() |
Assignee | |
Comment 4•11 years ago
|
||
Attachment #8400703 -
Flags: review?(matt.woodrow)
![]() |
Assignee | |
Updated•11 years ago
|
Attachment #8400230 -
Attachment is obsolete: true
Updated•11 years ago
|
Attachment #8400703 -
Flags: review?(matt.woodrow) → review+
![]() |
Assignee | |
Updated•11 years ago
|
Summary: [meta] Remove Image::DeprecatedGetAsSurface → Remove Image::DeprecatedGetAsSurface
![]() |
Assignee | |
Comment 5•11 years ago
|
||
OS: Mac OS X → All
Hardware: x86 → All
Version: 28 Branch → 31 Branch
Comment 6•11 years ago
|
||
Assignee: nobody → jwatt
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
![]() |
Assignee | |
Updated•11 years ago
|
Blocks: Moz2Dification
Updated•11 years ago
|
status-firefox31:
--- → affected
Whiteboard: [qa?]
Updated•11 years ago
|
Comment 7•11 years ago
|
||
jonathan, in comment 3 you seem to be saying that bug 989858 needs to be fixed before this can land. But bug 989858 isn't fixed and got untracked for 31. Is this OK to go into 31?
Is there any way I can test or verify this fix? Thanks !
Flags: needinfo?(jwatt)
![]() |
Assignee | |
Comment 8•11 years ago
|
||
(In reply to Liz Henry :lizzard from comment #7)
> Is there any way I can test or verify this fix? Thanks !
There's no reason to uplift this bugs patch, and it doesn't need qa attention. Thanks.
Flags: needinfo?(jwatt)
You need to log in
before you can comment on or make changes to this bug.
Description
•