Closed Bug 1028825 Opened 7 years ago Closed 7 years ago

Get rid of the Thebes backed gfxContext in Android's nsWindow.cpp

Categories

(Core :: Widget: Android, defect)

All
Android
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla33

People

(Reporter: jwatt, Assigned: jwatt)

References

Details

Attachments

(1 file)

Get rid of the Thebes backed gfxContext in Android's nsWindow.cpp.
Blocks: 933019
Seems like the nsWindow::DrawTo() methods are dead code. Try doesn't mind me removing them.
Component: Widget → Widget: Android
Attached patch patchSplinter Review
Attachment #8444280 - Flags: review?(chrislord.net)
Comment on attachment 8444280 [details] [diff] [review]
patch

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

Green try build and manual verification, LGTM.
Attachment #8444280 - Flags: review?(chrislord.net) → review+
Whoops, looks like I missed this code when I landed bug 1018980. Thanks for killing it :)
Depends on: 1018980
Comment on attachment 8444280 [details] [diff] [review]
patch

https://hg.mozilla.org/integration/mozilla-inbound/rev/31bf8c4b3e19

(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #4)
> Whoops, looks like I missed this code when I landed bug 1018980. Thanks for
> killing it :)

No probs. :)
Attachment #8444280 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/31bf8c4b3e19
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.