Reland PaintChunk::id

Reland PaintChunk::id

The id will be used in rasterization paint invalidation to match old
and new paint chunks. It can be also used in incremental layerization.

Original review url: https://codereview.chromium.org/2116693002

BUG=510908, 629946
Review-Url: https://codereview.chromium.org/2116693002
Committed: https://crrev.com/a1101d498d5054ecda5c4f6481b4738d7873b4da
Cr-Original-Commit-Position: refs/heads/master@{#406461}
Cr-Commit-Position: refs/heads/master@{#406793}

[Xianzhu, 2016-07-20 20:34:51]

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

[commit-bot: I haz the power, 2016-07-20 20:35:16]

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

[Xianzhu, 2016-07-20 20:35:38]

Description was changed from

==========
Reland PaintChunk::id

The id will be used in rasterization paint invalidation to match old
and new paint chunks. It can be also used in incremental layerization.

BUG=510908

Review-Url: https://codereview.chromium.org/2116693002
Cr-Commit-Position: refs/heads/master@{#406461}
==========

to

==========
Reland PaintChunk::id

The id will be used in rasterization paint invalidation to match old
and new paint chunks. It can be also used in incremental layerization.

BUG=510908,629946

Review-Url: https://codereview.chromium.org/2116693002
Cr-Commit-Position: refs/heads/master@{#406461}
==========

[Xianzhu, 2016-07-20 20:52:42]

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

[Xianzhu, 2016-07-20 20:52:43]

The only change compared to the original CL is to disable the tests mysteriously
failed on Windows debug bots.

[pdr., 2016-07-20 21:00:40]

On 2016/07/20 at 20:52:43, wangxianzhu wrote:
> The only change compared to the original CL is to disable the tests
mysteriously failed on Windows debug bots.

LGTM

[pdr., 2016-07-20 21:01:18]

Description was changed from

==========
Reland PaintChunk::id

The id will be used in rasterization paint invalidation to match old
and new paint chunks. It can be also used in incremental layerization.

BUG=510908,629946

Review-Url: https://codereview.chromium.org/2116693002
Cr-Commit-Position: refs/heads/master@{#406461}
==========

to

==========
Reland PaintChunk::id

The id will be used in rasterization paint invalidation to match old
and new paint chunks. It can be also used in incremental layerization.

Original review url: https://codereview.chromium.org/2116693002

BUG=510908,629946

Review-Url: https://codereview.chromium.org/2116693002
Cr-Commit-Position: refs/heads/master@{#406461}
==========

[pdr., 2016-07-20 21:01:18]

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

[pdr., 2016-07-20 21:01:34]

On 2016/07/20 at 21:00:40, pdr. wrote:
> On 2016/07/20 at 20:52:43, wangxianzhu wrote:
> > The only change compared to the original CL is to disable the tests
mysteriously failed on Windows debug bots.
> 
> LGTM

(I also added the original review url to the change description)

[chrishtr, 2016-07-20 22:10:21]

Let me try to repro this on a Windows machine I have here first.

[Xianzhu, 2016-07-20 22:14:56]

On 2016/07/20 22:10:21, chrishtr wrote:
> Let me try to repro this on a Windows machine I have here first.

I couldn't reproduce on my Windows 10 machine. Perhaps we need a Windows 7?

[commit-bot: I haz the power, 2016-07-20 23:33:19]

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

[commit-bot: I haz the power, 2016-07-20 23:33:20]

Dry run: This issue passed the CQ dry run.

[Xianzhu, 2016-07-21 00:53:54]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2016-07-21 00:54:19]

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

[Xianzhu, 2016-07-21 00:54:41]

The CQ bit was unchecked by wangxianzhu@chromium.org

[Xianzhu, 2016-07-21 01:01:35]

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

[commit-bot: I haz the power, 2016-07-21 01:02:40]

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-07-21 01:30:37]

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

[commit-bot: I haz the power, 2016-07-21 01:30:39]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Xianzhu, 2016-07-21 05:45:41]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-07-21 05:45:41]

The patchset sent to the CQ was uploaded after l-g-t-m from pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/2168513005/#ps40001
(title: "git cl try -b win_blink_dbg")

[commit-bot: I haz the power, 2016-07-21 05:46:02]

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-07-21 07:08:57]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-21 07:12:59]

Description was changed from

==========
Reland PaintChunk::id

The id will be used in rasterization paint invalidation to match old
and new paint chunks. It can be also used in incremental layerization.

Original review url: https://codereview.chromium.org/2116693002

BUG=510908,629946

Review-Url: https://codereview.chromium.org/2116693002
Cr-Commit-Position: refs/heads/master@{#406461}
==========

to

==========
Reland PaintChunk::id

The id will be used in rasterization paint invalidation to match old
and new paint chunks. It can be also used in incremental layerization.

Original review url: https://codereview.chromium.org/2116693002

BUG=510908,629946

Review-Url: https://codereview.chromium.org/2116693002
Committed: https://crrev.com/a1101d498d5054ecda5c4f6481b4738d7873b4da
Cr-Original-Commit-Position: refs/heads/master@{#406461}
Cr-Commit-Position: refs/heads/master@{#406793}
==========

[commit-bot: I haz the power, 2016-07-21 07:13:01]

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