Modify paint/invalidation/compositing/scrolling-neg-z-index-descendants-should-cause-repaint-expect…

Modify paint/invalidation/compositing/scrolling-neg-z-index-descendants-should-cause-repaint-expected.html to show bug

Previously the test was written in a way to hide the bug. It only
checked paint invalidation of the scrollbar. We miss paint invalidation
of the contents.

Convert the test into a text-based-repaint test, and move it to
paint/invalidation/compositing.

BUG=702006
Review-Url: https://codereview.chromium.org/2747333006
Cr-Commit-Position: refs/heads/master@{#457583}
Committed: https://chromium.googlesource.com/chromium/src/+/5db93ce184722588b24a21fc2f9c53b16cf9a45b

[Xianzhu, 2017-03-15 23:42:29]

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

[commit-bot: I haz the power, 2017-03-15 23:42:59]

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

[Xianzhu, 2017-03-15 23:47:59]

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

[commit-bot: I haz the power, 2017-03-16 00:44:34]

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

[commit-bot: I haz the power, 2017-03-16 00:44:34]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[wkorman, 2017-03-16 18:33:27]

lgtm

[Xianzhu, 2017-03-16 20:05:57]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2017-03-16 20:05:57]

The patchset sent to the CQ was uploaded after l-g-t-m from wkorman@chromium.org
Link to the patchset: https://codereview.chromium.org/2747333006/#ps20001
(title: "Correct TestExpectations")

[commit-bot: I haz the power, 2017-03-16 20:06:36]

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, 2017-03-16 21:14:21]

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

[commit-bot: I haz the power, 2017-03-16 21:14:23]

Failed to apply patch for third_party/WebKit/LayoutTests/TestExpectations:
While running git apply --index -p1;
  error: patch failed: third_party/WebKit/LayoutTests/TestExpectations:156
  error: third_party/WebKit/LayoutTests/TestExpectations: patch does not apply

Patch:       third_party/WebKit/LayoutTests/TestExpectations
Index: third_party/WebKit/LayoutTests/TestExpectations
diff --git a/third_party/WebKit/LayoutTests/TestExpectations
b/third_party/WebKit/LayoutTests/TestExpectations
index
4a689f97922c74ddfc9d37db21330af92105d65b..3a36383994b2f607a4bec365ac5f7822daa8c5ca
100644
--- a/third_party/WebKit/LayoutTests/TestExpectations
+++ b/third_party/WebKit/LayoutTests/TestExpectations
@@ -156,6 +156,9 @@ crbug.com/667045
virtual/disable-spinvalidation/paint/invalidation/table/composi
 
 crbug.com/701800 [ Win7 ]
virtual/disable-spinvalidation/compositing/perpendicular-layer-sorting.html [
Pass Failure ]
 
+crbug.com/702006
paint/invalidation/compositing/scrolling-neg-z-index-descendants-should-cause-repaint.html
[ Failure ]
+crbug.com/702006
virtual/disable-spinvalidation/paint/invalidation/compositing/scrolling-neg-z-index-descendants-should-cause-repaint.html
[ Failure ]
+
 # ====== Paint team owned tests to here ======
 
 crbug.com/702148 http/tests/images/gif-animated-partial-load.html [
NeedsRebaseline ]

[Xianzhu, 2017-03-16 21:16:56]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2017-03-16 21:16:56]

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

[commit-bot: I haz the power, 2017-03-16 21:17:38]

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, 2017-03-16 22:20:30]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1489699016085580,
"parent_rev": "94875b3b4d4a1fc047b99214f30078f3780259f0", "commit_rev":
"5db93ce184722588b24a21fc2f9c53b16cf9a45b"}

[commit-bot: I haz the power, 2017-03-16 22:21:11]

Description was changed from

==========
Modify
paint/invalidation/compositing/scrolling-neg-z-index-descendants-should-cause-repaint-expected.html
to show bug

Previously the test was written in a way to hide the bug. It only
checked paint invalidation of the scrollbar. We miss paint invalidation
of the contents.

Convert the test into a text-based-repaint test, and move it to
paint/invalidation/compositing.

BUG=702006
==========

to

==========
Modify
paint/invalidation/compositing/scrolling-neg-z-index-descendants-should-cause-repaint-expected.html
to show bug

Previously the test was written in a way to hide the bug. It only
checked paint invalidation of the scrollbar. We miss paint invalidation
of the contents.

Convert the test into a text-based-repaint test, and move it to
paint/invalidation/compositing.

BUG=702006

Review-Url: https://codereview.chromium.org/2747333006
Cr-Commit-Position: refs/heads/master@{#457583}
Committed:
https://chromium.googlesource.com/chromium/src/+/5db93ce184722588b24a21fc2f9c...
==========

[commit-bot: I haz the power, 2017-03-16 22:21:12]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/5db93ce184722588b24a21fc2f9c...