Make --enable-slimming-paint-v2 enable --enable-layer-lists

Make --enable-slimming-paint-v2 enable --enable-layer-lists

With this patch, enabling SlimmingPaintV2 will automatically enable
enable-layer-lists.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
BUG=630691
Committed: https://crrev.com/4cd50edf73839959f775daa562e209d3802dd64f
Cr-Commit-Position: refs/heads/master@{#415508}

[pdr., 2016-08-30 03:46:20]

The CQ bit was checked by pdr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-30 03:49:05]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-30 05:07:20]

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

[commit-bot: I haz the power, 2016-08-30 05:07:21]

Dry run: This issue passed the CQ dry run.

[pdr., 2016-08-30 17:20:41]

Description was changed from

==========
Make --enable-layer-lists imply --enable-slimming-paint-v2

With this patch, enabling SlimmingPaintV2 will automatically enable
enable-layer-lists.

BUG=630691
==========

to

==========
Make --enable-layer-lists imply --enable-slimming-paint-v2

With this patch, enabling SlimmingPaintV2 will automatically enable
enable-layer-lists.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v
BUG=630691
==========

[pdr., 2016-08-30 17:20:41]

pdr@chromium.org changed reviewers:
+ chrishtr@chromium.org, trchen@chromium.org

[pdr., 2016-08-30 17:20:52]

The CQ bit was checked by pdr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-30 17:21:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-30 17:22:51]

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

[commit-bot: I haz the power, 2016-08-30 17:22:52]

Dry run: Try jobs failed on following builders:
  linux_layout_tests_slimming_paint_v on master.tryserver.chromium.linux
(JOB_FAILED, no build URL)

[pdr., 2016-08-30 17:25:38]

Description was changed from

==========
Make --enable-layer-lists imply --enable-slimming-paint-v2

With this patch, enabling SlimmingPaintV2 will automatically enable
enable-layer-lists.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v
BUG=630691
==========

to

==========
Make --enable-layer-lists imply --enable-slimming-paint-v2

With this patch, enabling SlimmingPaintV2 will automatically enable
enable-layer-lists.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
BUG=630691
==========

[pdr., 2016-08-30 17:31:48]

https://codereview.chromium.org/2286343003/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2
(left):

https://codereview.chromium.org/2286343003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2:475:
compositing/video/video-reflection.html [ Failure ]
This was a mistake in
https://crrev.com/a52eb4201de9237701af410cb1d5c38a88373e58, just tidying it up
here.

[pdr., 2016-08-30 17:53:35]

The CQ bit was checked by pdr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-30 17:54:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[pdr., 2016-08-30 18:18:04]

The CQ bit was checked by pdr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-30 18:18:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[pdr., 2016-08-30 18:44:15]

The CQ bit was checked by pdr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-30 18:44:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chrishtr, 2016-08-30 20:32:40]

lgtm

[pdr., 2016-08-30 20:41:59]

The CQ bit was unchecked by pdr@chromium.org

[pdr., 2016-08-30 20:42:01]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2016-08-30 20:42:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-30 20:47:45]

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

[commit-bot: I haz the power, 2016-08-30 20:47:46]

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

[pdr., 2016-08-30 20:50:11]

pdr@chromium.org changed reviewers:
+ sievers@chromium.org

[pdr., 2016-08-30 20:50:43]

sievers, could you please review this with your content/OWNERS superpowers?

[no sievers, 2016-08-30 20:55:01]

lgtm

https://codereview.chromium.org/2286343003/diff/80001/content/browser/rendere...
File content/browser/renderer_host/render_process_host_impl.cc (right):

https://codereview.chromium.org/2286343003/diff/80001/content/browser/rendere...
content/browser/renderer_host/render_process_host_impl.cc:1482:
switches::kEnableSlimmingPaintV2))
nit: curly braces since it's multiline

[trchen, 2016-08-30 21:41:14]

The CL title doesn't match CL desc. Other than that lgtm. Yay!

[pdr., 2016-08-30 22:31:05]

https://codereview.chromium.org/2286343003/diff/80001/content/browser/rendere...
File content/browser/renderer_host/render_process_host_impl.cc (right):

https://codereview.chromium.org/2286343003/diff/80001/content/browser/rendere...
content/browser/renderer_host/render_process_host_impl.cc:1482:
switches::kEnableSlimmingPaintV2))
On 2016/08/30 at 20:55:01, sievers wrote:
> nit: curly braces since it's multiline

Good idea, done.

[pdr., 2016-08-30 22:32:48]

Description was changed from

==========
Make --enable-layer-lists imply --enable-slimming-paint-v2

With this patch, enabling SlimmingPaintV2 will automatically enable
enable-layer-lists.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
BUG=630691
==========

to

==========
Make --enable-slimming-paint-v2 enable --enable-layer-lists

With this patch, enabling SlimmingPaintV2 will automatically enable
enable-layer-lists.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
BUG=630691
==========

[pdr., 2016-08-30 22:33:27]

On 2016/08/30 at 21:41:14, trchen wrote:
> The CL title doesn't match CL desc. Other than that lgtm. Yay!

Good catch, I switched this to "Make --enable-slimming-paint-v2 enable
--enable-layer-lists".

[pdr., 2016-08-30 22:33:30]

The CQ bit was checked by pdr@chromium.org

[pdr., 2016-08-30 22:33:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrishtr@chromium.org, sievers@chromium.org, trchen@chromium.org
Link to the patchset: https://codereview.chromium.org/2286343003/#ps100001
(title: "Braces!")

[commit-bot: I haz the power, 2016-08-30 22:34:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-31 00:17:43]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-08-31 00:21:14]

Description was changed from

==========
Make --enable-slimming-paint-v2 enable --enable-layer-lists

With this patch, enabling SlimmingPaintV2 will automatically enable
enable-layer-lists.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
BUG=630691
==========

to

==========
Make --enable-slimming-paint-v2 enable --enable-layer-lists

With this patch, enabling SlimmingPaintV2 will automatically enable
enable-layer-lists.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
BUG=630691

Committed: https://crrev.com/4cd50edf73839959f775daa562e209d3802dd64f
Cr-Commit-Position: refs/heads/master@{#415508}
==========

[commit-bot: I haz the power, 2016-08-31 00:21:15]

Patchset 6 (id:??) landed as
https://crrev.com/4cd50edf73839959f775daa562e209d3802dd64f
Cr-Commit-Position: refs/heads/master@{#415508}