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

Issue 13800013: Disable shader disk cache by default. (Closed)

Created:
7 years, 8 months ago by dsinclair
Modified:
7 years, 8 months ago
Reviewers:
piman
CC:
chromium-reviews, nkostylev+watch_chromium.org, jam, apatrick_chromium, joi+watch-content_chromium.org, darin-cc_chromium.org, oshima+watch_chromium.org, stevenjb+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

Disable shader disk cache by default. The code to handle cache clearing didn't land in time for the M27 branch so it has been decided to disable this feature for M27. This patch will be rolled back after it lands on the M27 branch. BUG=228989 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192949

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -44 lines) Patch
M chrome/browser/chromeos/login/chrome_restart_request.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/gpu/gpu_process_host.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M gpu/command_buffer/service/gpu_switches.h View 1 chunk +1 line, -1 line 0 comments Download
M gpu/command_buffer/service/gpu_switches.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M gpu/command_buffer/service/memory_program_cache.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M gpu/command_buffer/service/memory_program_cache_unittest.cc View 1 4 chunks +0 lines, -36 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
dsinclair
piman, PTAL.
7 years, 8 months ago (2013-04-08 19:57:17 UTC) #1
piman
lgtm
7 years, 8 months ago (2013-04-08 20:16:21 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dsinclair@chromium.org/13800013/1
7 years, 8 months ago (2013-04-08 20:17:59 UTC) #3
commit-bot: I haz the power
Retried try job too often on linux_rel for step(s) gpu_unittests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&number=115044
7 years, 8 months ago (2013-04-08 21:00:58 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dsinclair@chromium.org/13800013/14003
7 years, 8 months ago (2013-04-08 21:04:52 UTC) #5
commit-bot: I haz the power
7 years, 8 months ago (2013-04-08 23:36:45 UTC) #6
Message was sent while issue was closed.
Change committed as 192949

Powered by Google App Engine
This is Rietveld 408576698