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

Issue 62093002: Revert 230444 "content: Disable CommandBufferProxyImpl::Flush tr..." (Closed)

Created:
7 years, 1 month ago by reveman
Modified:
7 years, 1 month ago
Reviewers:
nduca, reveman, piman
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 230444 "content: Disable CommandBufferProxyImpl::Flush tr..." > content: Disable CommandBufferProxyImpl::Flush traces by default. > > BUG=309441 > > Review URL: https://codereview.chromium.org/33003003 The number of command buffer flushes with impl-side painting should now have been reduced to an amount that allows us to have these traces enabled by default. TBR=piman@chromium.org BUG=309441 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233293

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M trunk/src/content/common/gpu/client/command_buffer_proxy_impl.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
reveman
7 years, 1 month ago (2013-11-06 14:56:44 UTC) #1
reveman
I prefer to have these traces enabled by default to make it easier to spot ...
7 years, 1 month ago (2013-11-06 15:01:14 UTC) #2
reveman
7 years, 1 month ago (2013-11-06 17:50:45 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r233293 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698