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

Issue 8637014: Plumb thru that we cache the front buffer. (Closed)

Created:
9 years ago by jonathan.backer
Modified:
9 years ago
Reviewers:
apatrick_chromium
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium, dpranke-watch+content_chromium.org, piman, marcheu
Visibility:
Public.

Description

Plumb thru that we cache the front buffer. This allows us to drop the root layer textures and save on VRAM (10 MB on a 30" display). WebKit side patch: https://bugs.webkit.org/show_bug.cgi?id=72956 BUG=none TEST='cat /proc/meminfo | grep Shmem' on Intel graphics and use_aura=1 attached to 30" display Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111532

Patch Set 1 #

Patch Set 2 : Merge #

Patch Set 3 : Merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+68 lines, -12 lines) Patch
M content/common/gpu/image_transport_surface_linux.cc View 1 7 chunks +19 lines, -5 lines 0 comments Download
M content/common/gpu/image_transport_surface_mac.cc View 1 3 chunks +15 lines, -2 lines 0 comments Download
M content/common/gpu/image_transport_surface_win.cc View 1 2 chunks +20 lines, -5 lines 0 comments Download
M gpu/command_buffer/service/feature_info.cc View 1 2 1 chunk +4 lines, -0 lines 0 comments Download
M third_party/khronos/GLES2/gl2ext.h View 1 2 1 chunk +10 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jonathan.backer
9 years ago (2011-11-22 15:03:02 UTC) #1
jonathan.backer
On 2011/11/22 15:03:02, jonathan.backer wrote: Did some testing. Dropping the root layer tiles can cost ...
9 years ago (2011-11-22 18:21:31 UTC) #2
apatrick_chromium
9 years ago (2011-11-23 01:48:35 UTC) #3
lgtm

Powered by Google App Engine
This is Rietveld 408576698