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

Issue 92103002: Don't check the framebuffer status every time we flush when we're on Chromium; (Closed)

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

Description

Don't check the framebuffer status every time we flush when we're on Chromium; this reduces the number of unnecessary command buffer flushes by about 50% for page loads when doing Ganesh rendering, since we do a flush on clear. BUG= Committed: http://code.google.com/p/skia/source/detail?r=12421

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -4 lines) Patch
M src/gpu/gl/GrGpuGL.cpp View 2 chunks +11 lines, -4 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
humper
ptal
7 years ago (2013-11-27 17:22:58 UTC) #1
bsalomon
lgtm
7 years ago (2013-11-27 17:48:01 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/humper@google.com/92103002/1
7 years ago (2013-11-27 18:01:41 UTC) #3
commit-bot: I haz the power
7 years ago (2013-11-27 18:13:20 UTC) #4
Message was sent while issue was closed.
Change committed as 12421

Powered by Google App Engine
This is Rietveld 408576698