Summary: | [chromium] Adding support for GL_EXT_discard_framebuffer. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Michal Mocny <mmocny> | ||||||
Component: | New Bugs | Assignee: | Michal Mocny <mmocny> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, cc-bugs, enne, fishd, jamesr, nduca, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Michal Mocny
2012-03-16 11:54:13 PDT
Created attachment 132338 [details]
Patch
Comment on attachment 132338 [details] Patch This is the other WebGraphicsContext3D extension needed for compositor memory allocation changed callback handler. (The other is https://bugs.webkit.org/show_bug.cgi?id=81279) Please wait for approval from fishd@chromium.org, abarth@webkit.org or jamesr@chromium.org before submitting because this patch contains changes to the Chromium platform API. Comment on attachment 132338 [details] Patch This cl will change, as the extension is going to have some arguments to better mirror http://www.khronos.org/registry/gles/extensions/EXT/EXT_discard_framebuffer.txt Created attachment 132587 [details]
Patch
Comment on attachment 132587 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=132587&action=review > Source/Platform/chromium/public/WebGraphicsContext3D.h:162 > + virtual void ensureFramebufferCHROMIUM() { } This is a chromium extension to the discard framebuffer extension which is very likely to be removed once we support the automatic recreation of backbuffer upon first use. Due to the time pressure for getting this in, it is unlikely to happen before M19, hence this explicit call. Is it alright to share the single "GL_EXT_discard_framebuffer" string, or shall I add a "GL_CHROMIUM_ensure_framebuffer"? Comment on attachment 132587 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=132587&action=review LGTM. @enne or @jamesr for review. >> Source/Platform/chromium/public/WebGraphicsContext3D.h:162 >> + virtual void ensureFramebufferCHROMIUM() { } > > This is a chromium extension to the discard framebuffer extension which is very likely to be removed once we support the automatic recreation of backbuffer upon first use. > Due to the time pressure for getting this in, it is unlikely to happen before M19, hence this explicit call. > > Is it alright to share the single "GL_EXT_discard_framebuffer" string, or shall I add a "GL_CHROMIUM_ensure_framebuffer"? Seems fine Comment on attachment 132587 [details] Patch Clearing flags on attachment: 132587 Committed r111312: <http://trac.webkit.org/changeset/111312> All reviewed patches have been landed. Closing bug. |