Closed
Bug 1348209
Opened 7 years ago
Closed 7 years ago
Make wr_renderer_readback() as to handle both GL and GLES
Categories
(Core :: Graphics: WebRender, enhancement)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: sotaro, Assigned: sotaro)
References
Details
Attachments
(1 file, 2 obsolete files)
2.07 KB,
patch
|
sotaro
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → sotaro.ikeda.g
Assignee | ||
Comment 1•7 years ago
|
||
Assignee | ||
Comment 2•7 years ago
|
||
The functions were borrowed from https://github.com/servo/webrender/pull/984.
Assignee | ||
Updated•7 years ago
|
Assignee | ||
Updated•7 years ago
|
Attachment #8848390 -
Flags: review?(bugmail)
Comment 3•7 years ago
|
||
Comment on attachment 8848390 [details] [diff] [review] patch - Make wr_renderer_readback() as to handle both GL and GLES Review of attachment 8848390 [details] [diff] [review]: ----------------------------------------------------------------- r+ but in the future I will r- any review request without a commit message, so please make sure to add commit messages going forward. ::: gfx/webrender_bindings/src/bindings.rs @@ +56,5 @@ > check_ffi_type!(_namespace_id_repr struct IdNamespace as (u32)); > > +const GL_FORMAT_BGRA_GL: gl::GLuint = gl::BGRA; > + > +const GL_FORMAT_BGRA_GLES: gl::GLuint = gl::BGRA_EXT; nit: remove the empty line between these two
Attachment #8848390 -
Flags: review?(bugmail) → review+
Assignee | ||
Comment 4•7 years ago
|
||
Attachment #8848390 -
Attachment is obsolete: true
Attachment #8849367 -
Flags: review+
Assignee | ||
Comment 5•7 years ago
|
||
Attachment #8849367 -
Attachment is obsolete: true
Attachment #8849368 -
Flags: review+
Assignee | ||
Comment 6•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2265e179245749c52c384c1e2be78453615d1ec2
Pushed by sikeda@mozilla.com: https://hg.mozilla.org/projects/graphics/rev/8680b14ea3d0 Make wr_renderer_readback() as to handle both GL and GLES r=kats
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment 8•7 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8680b14ea3d0
status-firefox55:
--- → fixed
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•