Using experiments to activate the DisplayList2DCanvas optimization

Using experiments to activate the DisplayList2DCanvas optimization

BUG=401187
Committed: https://crrev.com/bc9cd5f5dab93e2867c791b3512edc0f5f84184b
Cr-Commit-Position: refs/heads/master@{#291998}

[Rémi Piotaix, 2014-08-13 21:50:21]

PTAL

[jam, 2014-08-15 17:33:39]

rerouting to Avi

[Avi (use Gerrit), 2014-08-15 17:41:27]

lgtm

stampity stamp

[Rémi Piotaix, 2014-08-15 17:51:51]

On 2014/08/15 17:41:27, Avi wrote:
> lgtm
> 
> stampity stamp

stampity stamp ? :s

[Rémi Piotaix, 2014-08-15 18:07:06]

The CQ bit was checked by piotaixr@chromium.org

[commit-bot: I haz the power, 2014-08-15 18:08:58]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/piotaixr@chromium.org/447863003/20001

[commit-bot: I haz the power, 2014-08-15 22:04:28]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tri...)
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win_chromium_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-15 22:07:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-15 22:07:52]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Rémi Piotaix, 2014-08-21 19:18:19]

New patch: all test OK locally, added a line in histograms.xml

Still missing LGTM for files in chrome/browser/...

[Avi (use Gerrit), 2014-08-21 19:19:33]

On 2014/08/21 19:18:19, Rémi Piotaix wrote:
> New patch: all test OK locally, added a line in histograms.xml
> 
> Still missing LGTM for files in chrome/browser/...

Not only are you missing an LG for the chrome/browser files, you are missing a
reviewer for the histogram file.

[Rémi Piotaix, 2014-08-21 19:36:09]

On 2014/08/21 19:19:33, Avi wrote:
> On 2014/08/21 19:18:19, Rémi Piotaix wrote:
> > New patch: all test OK locally, added a line in histograms.xml
> > 
> > Still missing LGTM for files in chrome/browser/...
> 
> Not only are you missing an LG for the chrome/browser files, you are missing a
> reviewer for the histogram file.

Adding asvitkine@ for the histogram.xml file.

[Alexei Svitkine (slow), 2014-08-21 19:36:54]

lgtm

[Rémi Piotaix, 2014-08-26 20:13:25]

piotaixr@chromium.org changed reviewers:
+ alemate@chromium.org

[Rémi Piotaix, 2014-08-26 20:15:07]

On 2014/08/26 20:13:25, Rémi Piotaix wrote:
> mailto:piotaixr@chromium.org changed reviewers:
> + mailto:alemate@chromium.org

@alemate: PTAL to the chrome_restart_request.cc file (if i'm not mistaken, it is
the only one which has no LGTM)

[Alexander Alekseev, 2014-08-26 20:31:09]

lgtm

[Rémi Piotaix, 2014-08-26 20:49:15]

The CQ bit was checked by piotaixr@chromium.org

[commit-bot: I haz the power, 2014-08-26 20:50:10]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/piotaixr@chromium.org/447863003/40001

[commit-bot: I haz the power, 2014-08-26 22:24:59]

Committed patchset #3 (40001) as 99cde8e13b3a9bca77836528421ce97fc0514430

[commit-bot: I haz the power, 2014-09-10 02:46:16]

Patchset 3 (id:??) landed as
https://crrev.com/bc9cd5f5dab93e2867c791b3512edc0f5f84184b
Cr-Commit-Position: refs/heads/master@{#291998}