Remove Image::DeprecatedGetAsSurface

RESOLVED FIXED in Firefox 31

Status

()

defect
RESOLVED FIXED
5 years ago
2 months ago

People

(Reporter: ali, Assigned: jwatt)

Tracking

(Blocks 2 bugs)

31 Branch
mozilla31
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox31 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

5 years ago
Remove calls to Image::DeprecatedGetAsSurface and use Image::GetAsSourceSurface instead.
(Reporter)

Updated

5 years ago
Blocks: 947194
Depends on: 959121, 959120
(Reporter)

Updated

5 years ago
Depends on: 964731
(Reporter)

Updated

5 years ago
Depends on: 964732
(Reporter)

Updated

5 years ago
Depends on: 972843
Marking this as NEW based on the conversation in bug 947194.
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Updated

5 years ago
Depends on: 982395
(Assignee)

Updated

5 years ago
Depends on: 982419
(Assignee)

Updated

5 years ago
Depends on: 982427
(Assignee)

Updated

5 years ago
Depends on: 990752
(Assignee)

Comment 2

5 years ago
Posted patch patch (obsolete) — Splinter Review
The work in the various blocking bugs needs to be finished and then this should do the rest.
(Assignee)

Comment 3

5 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

5 years ago
Posted patch patchSplinter Review
Attachment #8400703 - Flags: review?(matt.woodrow)
(Assignee)

Updated

5 years ago
Attachment #8400230 - Attachment is obsolete: true
(Assignee)

Updated

5 years ago
Blocks: 950372
Attachment #8400703 - Flags: review?(matt.woodrow) → review+
(Assignee)

Updated

5 years ago
Depends on: 993784
(Assignee)

Updated

5 years ago
No longer depends on: 989858
(Assignee)

Updated

5 years ago
Summary: [meta] Remove Image::DeprecatedGetAsSurface → Remove Image::DeprecatedGetAsSurface
(Assignee)

Comment 5

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/f2b29dbe2ae9
OS: Mac OS X → All
Hardware: x86 → All
Version: 28 Branch → 31 Branch
https://hg.mozilla.org/mozilla-central/rev/f2b29dbe2ae9
Assignee: nobody → jwatt
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
(Assignee)

Updated

5 years ago
Whiteboard: [qa?]
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

5 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)
qa- as per comment 8.
QA Whiteboard: [qa-]
Whiteboard: [qa?]
You need to log in before you can comment on or make changes to this bug.