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

Issue 150403007: cc: Disable "activate to low res" optimization (Closed)

Created:
6 years, 10 months ago by enne (OOO)
Modified:
6 years, 10 months ago
Reviewers:
danakj
CC:
chromium-reviews, cc-bugs_chromium.org, aelias_OOO_until_Jul13, reveman
Visibility:
Public.

Description

cc: Disable "activate to low res" optimization This optimization is broken on m33 and causes flashing to tabs. Turning off the optimization fixes the problem. This is an intentionally small patch meant for merging to m33 and then immediately reverting on trunk. BUG=335289 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=250332

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -2 lines) Patch
M cc/layers/picture_layer_impl.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M cc/layers/picture_layer_impl_unittest.cc View 2 chunks +4 lines, -2 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
enne (OOO)
6 years, 10 months ago (2014-02-10 23:03:39 UTC) #1
danakj
LGTM
6 years, 10 months ago (2014-02-11 00:37:47 UTC) #2
enne (OOO)
The CQ bit was checked by enne@chromium.org
6 years, 10 months ago (2014-02-11 00:40:48 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/enne@chromium.org/150403007/1
6 years, 10 months ago (2014-02-11 00:44:56 UTC) #4
commit-bot: I haz the power
Change committed as 250332
6 years, 10 months ago (2014-02-11 04:14:26 UTC) #5
enne (OOO)
6 years, 10 months ago (2014-02-12 19:57:50 UTC) #6
Message was sent while issue was closed.
A revert of this CL has been created in
https://codereview.chromium.org/161023002/ by enne@chromium.org.

The reason for reverting is: Reverting temporary patch merged to m33..

Powered by Google App Engine
This is Rietveld 408576698