Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(407)

Issue 14461006: Plumbed in discard_framebuffer extension (Closed)

Created:
7 years, 7 months ago by robertphillips
Modified:
7 years, 7 months ago
Reviewers:
bsalomon
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

First step. Compiled and run on: Win7, MacPro, Linux and Android.

Patch Set 1 #

Patch Set 2 : Fixed on Linux #

Patch Set 3 : Fixed on Mac #

Patch Set 4 : Fixed Andoid #

Patch Set 5 : Fixed other issues #

Total comments: 4

Patch Set 6 : Removed desktop side changes & validation test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -2 lines) Patch
M include/gpu/gl/GrGLFunctions.h View 4 1 chunk +1 line, -0 lines 0 comments Download
M include/gpu/gl/GrGLInterface.h View 4 1 chunk +1 line, -0 lines 0 comments Download
M src/gpu/gl/GrGLCaps.h View 4 2 chunks +4 lines, -0 lines 0 comments Download
M src/gpu/gl/GrGLCaps.cpp View 4 4 chunks +5 lines, -0 lines 0 comments Download
M src/gpu/gl/GrGLCreateNullInterface.cpp View 4 1 chunk +1 line, -0 lines 0 comments Download
M src/gpu/gl/GrGLInterface.cpp View 1 2 3 4 5 1 chunk +9 lines, -0 lines 0 comments Download
M src/gpu/gl/GrGLNoOpInterface.h View 4 1 chunk +4 lines, -0 lines 0 comments Download
M src/gpu/gl/GrGLNoOpInterface.cpp View 4 1 chunk +5 lines, -0 lines 0 comments Download
M src/gpu/gl/GrGpuGL.cpp View 4 1 chunk +2 lines, -2 lines 0 comments Download
M src/gpu/gl/android/GrGLCreateNativeInterface_android.cpp View 1 2 3 4 1 chunk +3 lines, -0 lines 0 comments Download
M src/gpu/gl/debug/GrGLCreateDebugInterface.cpp View 4 1 chunk +1 line, -0 lines 0 comments Download
M src/gpu/gl/iOS/GrGLCreateNativeInterface_iOS.cpp View 1 2 3 4 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
robertphillips
Not used yet but enough changes that I don't want the bits to rot.
7 years, 7 months ago (2013-04-29 14:35:08 UTC) #1
bsalomon
https://codereview.chromium.org/14461006/diff/16001/src/gpu/gl/GrGLInterface.cpp File src/gpu/gl/GrGLInterface.cpp (right): https://codereview.chromium.org/14461006/diff/16001/src/gpu/gl/GrGLInterface.cpp#newcode298 src/gpu/gl/GrGLInterface.cpp:298: if (extensions.has("GL_EXT_discard_framebuffer")) { Might be worth checking if Chrome ...
7 years, 7 months ago (2013-04-29 14:42:15 UTC) #2
robertphillips
This patch removes the desktop-side handling for the discard extension and disables the validation check ...
7 years, 7 months ago (2013-04-29 15:25:38 UTC) #3
bsalomon
lgtm
7 years, 7 months ago (2013-04-29 16:40:58 UTC) #4
robertphillips
7 years, 7 months ago (2013-04-29 16:50:38 UTC) #5
Message was sent while issue was closed.
committed as r8906

Powered by Google App Engine
This is Rietveld 408576698