Invalidate only non-scrolling layers for non-scrolling content.

Invalidate only non-scrolling layers for non-scrolling content.

BUG=416535
Committed: https://crrev.com/59fd991c44d867f1121b1a0c1e0aaaad8c440760
Cr-Commit-Position: refs/heads/master@{#397912}

[chrishtr, 2016-06-03 16:33:02]

Description was changed from

==========
none

BUG=
==========

to

==========
Invalidate only non-scrolling layers for non-scrolling content.

BUG=416535
==========

[chrishtr, 2016-06-03 18:25:52]

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

[chrishtr, 2016-06-03 18:25:52]

See PS1 for expected rebaseline results.

[Xianzhu, 2016-06-03 18:36:11]

On 2016/06/03 18:25:52, chrishtr wrote:
> See PS1 for expected rebaseline results.

In PS1 the actual pixel result of the non-compositing test case also changed.
Does PS2 produce the same result?

[chrishtr, 2016-06-03 22:43:45]

On 2016/06/03 at 18:36:11, wangxianzhu wrote:
> On 2016/06/03 18:25:52, chrishtr wrote:
> > See PS1 for expected rebaseline results.
> 
> In PS1 the actual pixel result of the non-compositing test case also changed.
Does PS2 produce the same result?

Good catch. That was a real bug. Fixed. The problem was that it's only safe to
differentiate between scrolling and non-scrolling layers
for composited-scrolling content. Even without composited scrolling, there may
be a foreground layer.

[Xianzhu, 2016-06-03 22:52:52]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-06-03 22:52:53]

lgtm

Pixel expectations of repaint tests did the job :)

lgtm.

[commit-bot: I haz the power, 2016-06-03 22:53:41]

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

[commit-bot: I haz the power, 2016-06-03 23:14:39]

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

[commit-bot: I haz the power, 2016-06-03 23:14:40]

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

[chrishtr, 2016-06-04 06:21:53]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2016-06-04 06:21:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
wangxianzhu@chromium.org
Link to the patchset: https://codereview.chromium.org/2039503002/#ps100001
(title: " ")

[commit-bot: I haz the power, 2016-06-04 06:22:11]

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

[commit-bot: I haz the power, 2016-06-04 11:08:32]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-06-04 11:10:08]

Description was changed from

==========
Invalidate only non-scrolling layers for non-scrolling content.

BUG=416535
==========

to

==========
Invalidate only non-scrolling layers for non-scrolling content.

BUG=416535

Committed: https://crrev.com/59fd991c44d867f1121b1a0c1e0aaaad8c440760
Cr-Commit-Position: refs/heads/master@{#397912}
==========

[commit-bot: I haz the power, 2016-06-04 11:10:09]

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