Enable slimming paint for fast/repaint

Enable slimming paint for fast/repaint

BUG=497950
R=chrishtr@chromium.org, wkorman@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197198

[Xianzhu, 2015-06-16 19:53:45]

wangxianzhu@chromium.org changed reviewers:
+ chrishtr@chromium.org, wkorman@chromium.org

[Xianzhu, 2015-06-16 19:53:54]

This is not actually reviewable because of the huge size of the patch. The tab
will be stuck for some time before the CL can be displayed.

[wkorman, 2015-06-16 20:04:06]

lgtm

It looks like we're just moving the expectations from virtual to non-virtual,
and those were already reviewed, right?

Do we need to modify the virtual/slimmingpaint/fast/repaint entry in
VirtualTestSuites or are we keeping it around in interim?

[Xianzhu, 2015-06-16 20:29:16]

On Tue, Jun 16, 2015 at 1:04 PM, <wkorman@chromium.org> wrote:

> lgtm
>
> It looks like we're just moving the expectations from virtual to
> non-virtual,
> and those were already reviewed, right?
>

Yes.


>
> Do we need to modify the virtual/slimmingpaint/fast/repaint entry in
> VirtualTestSuites or are we keeping it around in interim?
>
>
Currently we registered the whole virtual/slimmingpaint/fast which will be
removed after we migrate all fast/ tests. We could enable slimming paint
the whole fast/ directory but the CL would be too big to manage.
fast/repaint might be the sub-directory needing the biggest CL (because of
the extended format of repaint results for slimming paint) so I migrate it
first. Perhaps the rest of fast/ could be just another single CL.


> https://codereview.chromium.org/1185423002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[chrishtr, 2015-06-16 20:33:06]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2015-06-16 20:33:12]

lgtm

[commit-bot: I haz the power, 2015-06-16 20:34:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1185423002/10001

[commit-bot: I haz the power, 2015-06-16 20:59:40]

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

[commit-bot: I haz the power, 2015-06-16 20:59:47]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/3...)

[Xianzhu, 2015-06-16 21:17:00]

Committed patchset #2 (id:10001) manually as 197198 (presubmit successful).