Closed Bug 1255303 Opened 9 years ago Closed 9 years ago

Use SurfaceFormat::B8G8R8X8 in BasicCompositor::CreateRenderTarget() if possible

Categories

(Core :: Graphics: Layers, defect)

Product:
Core
Component:
Graphics: Layers
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla48
Tracking Flags:
Tracking Status
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: sotaro, Assigned: sotaro)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 4 obsolete files)

patch - Use SurfaceFormat::B8G8R8X8 as back buffer if possible
(deleted), patch
sotaro
: review+
Details | Diff | Splinter Review
BasicCompositor::CreateRenderTarget() always create draw target as SurfaceFormat::B8G8R8A8.
Blocks: unaccel-video
Assignee: nobody → sotaro.ikeda.g
Blocks: 1254897
With talking with jeff, we are going to move CreateRenderTarget() for window to widget.
(In reply to Sotaro Ikeda [:sotaro] from comment #3) > With talking with jeff, we are going to move CreateRenderTarget() for window > to widget. I changed my mind. BasicCompositor composites not only to widget's screen, but also for making Snapshot. To handle both cases, it seems better to keep the code in BasicCompositor in this bug.
Attachment #8728807 - Attachment is obsolete: true
Attachment #8730489 - Flags: review?(jmuizelaar)
(In reply to Sotaro Ikeda [:sotaro] from comment #4) > (In reply to Sotaro Ikeda [:sotaro] from comment #3) > > With talking with jeff, we are going to move CreateRenderTarget() for window > > to widget. > > I changed my mind. BasicCompositor composites not only to widget's screen, > but also for making Snapshot. To handle both cases, it seems better to keep > the code in BasicCompositor in this bug. What do you still plan to move things into widget? I don't think we want to be doing our weird DIBSection as an image surface business in CreateBackBufferRenderTargetForWindow.
Okey, I am going to include it in a next patch.
Attachment #8730489 - Flags: review?(jmuizelaar)
(In reply to Jeff Muizelaar [:jrmuizel] from comment #7) > What do you still plan to move things into widget? I don't think we want to > be doing our weird DIBSection as an image surface business in > CreateBackBufferRenderTargetForWindow. Yes, I am going to plan DIBSection thing in widget. I just put off the thing to another bugs like Bug 1254897 and Bug 1255703.
Attachment #8730489 - Attachment is obsolete: true
attachment 8731089 [details] [diff] [review] moved a back buffer allocation to widget.
Attachment #8731089 - Flags: review?(jmuizelaar)
update nits.
Attachment #8731089 - Attachment is obsolete: true
Attachment #8731089 - Flags: review?(jmuizelaar)
Attachment #8731092 - Flags: review?(jmuizelaar)
Attachment #8731092 - Flags: review?(jmuizelaar) → review+
https://hg.mozilla.org/mozilla-central/rev/d916e4520180
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
It causes Bug 1258094.
No longer blocks: 1254897
Depends on: 1255703
Rebased.
Attachment #8731092 - Attachment is obsolete: true
Attachment #8752627 - Flags: review+
Going to re-check in since bug 1255703 is addressed.
https://hg.mozilla.org/mozilla-central/rev/76361667f62b
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: