Fix flaky test paint/invalidation/background-resize-height.html

Fix flaky test paint/invalidation/background-resize-height.html

Add capturePixelsThenAsync() to ensure image decoding is finished before
running the test, to avoid flakiness of "contentsOpaque" in the dumped
layer tree.

Also disabled the fixed height gradient case because it produces
invisible pixel difference.

BUG=658479
Committed: https://crrev.com/586f1b550af51392707deba2b9a1c9aaeeeb8bc9
Cr-Commit-Position: refs/heads/master@{#429506}

[Xianzhu, 2016-11-01 23:05:57]

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

[commit-bot: I haz the power, 2016-11-01 23:06:37]

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

[Xianzhu, 2016-11-01 23:07:13]

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

[chrishtr, 2016-11-01 23:20:20]

https://codereview.chromium.org/2464253002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/paint/invalidation/background-resize-height.html
(right):

https://codereview.chromium.org/2464253002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/paint/invalidation/background-resize-height.html:96:
<!-- This produces invisible pixel difference.
Just delete this div then?

[Xianzhu, 2016-11-01 23:59:19]

https://codereview.chromium.org/2464253002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/paint/invalidation/background-resize-height.html
(right):

https://codereview.chromium.org/2464253002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/paint/invalidation/background-resize-height.html:96:
<!-- This produces invisible pixel difference.
On 2016/11/01 23:20:20, chrishtr wrote:
> Just delete this div then?

Done locally. Will upload after rebaseline-cl.

[commit-bot: I haz the power, 2016-11-02 00:42:06]

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

[commit-bot: I haz the power, 2016-11-02 00:42:07]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Xianzhu, 2016-11-02 23:27:19]

On 2016/11/01 23:59:19, Xianzhu wrote:
>
https://codereview.chromium.org/2464253002/diff/1/third_party/WebKit/LayoutTe...
> File
>
third_party/WebKit/LayoutTests/paint/invalidation/background-resize-height.html
> (right):
> 
>
https://codereview.chromium.org/2464253002/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/paint/invalidation/background-resize-height.html:96:
> <!-- This produces invisible pixel difference.
> On 2016/11/01 23:20:20, chrishtr wrote:
> > Just delete this div then?
> 
> Done locally. Will upload after rebaseline-cl.

Done.

[Xianzhu, 2016-11-02 23:27:37]

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

[commit-bot: I haz the power, 2016-11-02 23:28:21]

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-11-03 00:26:44]

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

[commit-bot: I haz the power, 2016-11-03 00:26:45]

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_...)

[chrishtr, 2016-11-03 00:41:04]

lgtm

[chrishtr, 2016-11-03 00:41:08]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2016-11-03 00:41:36]

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

[Xianzhu, 2016-11-03 00:42:00]

The CQ bit was unchecked by wangxianzhu@chromium.org

[Xianzhu, 2016-11-03 00:43:05]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-11-03 00:43:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrishtr@chromium.org
Link to the patchset: https://codereview.chromium.org/2464253002/#ps40001
(title: "Remove extra files (should belong to another CL)")

[commit-bot: I haz the power, 2016-11-03 00:43:29]

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-11-03 01:47:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-03 01:49:24]

Description was changed from

==========
Fix flaky test paint/invalidation/background-resize-height.html

Add capturePixelsThenAsync() to ensure image decoding is finished before
running the test, to avoid flakiness of "contentsOpaque" in the dumped
layer tree.

Also disabled the fixed height gradient case because it produces
invisible pixel difference.

BUG=658479
==========

to

==========
Fix flaky test paint/invalidation/background-resize-height.html

Add capturePixelsThenAsync() to ensure image decoding is finished before
running the test, to avoid flakiness of "contentsOpaque" in the dumped
layer tree.

Also disabled the fixed height gradient case because it produces
invisible pixel difference.

BUG=658479

Committed: https://crrev.com/586f1b550af51392707deba2b9a1c9aaeeeb8bc9
Cr-Commit-Position: refs/heads/master@{#429506}
==========

[commit-bot: I haz the power, 2016-11-03 01:49:25]

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