Remove Framebuffer.cpp

RESOLVED FIXED in mozilla34

Status

()

Core
Widget: Gonk
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mwu, Assigned: mwu)

Tracking

unspecified
mozilla34
All
Gonk (Firefox OS)
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Created attachment 8461108 [details] [diff] [review]
Remove Framebuffer.cpp

We only have Framebuffer.cpp for querying the FB size, but there are better ways of doing that these days.

The change in nsWindow::nsWindow() is a bit messy - I cleaned up the indentation in there at the same time, but the only real change is in how we obtain the fb size.
Attachment #8461108 - Flags: review?(sotaro.ikeda.g)
Comment on attachment 8461108 [details] [diff] [review]
Remove Framebuffer.cpp

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

Nice!
Attachment #8461108 - Flags: review?(sotaro.ikeda.g) → review+
(Assignee)

Comment 2

3 years ago
https://hg.mozilla.org/integration/b2g-inbound/rev/a0d61b6bb680
This merged to m-c last night:

https://hg.mozilla.org/mozilla-central/rev/a0d61b6bb680
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.