Increment property tree sequence number when updating.

Increment property tree sequence number when updating.

BUG=692842
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2725513002
Cr-Commit-Position: refs/heads/master@{#453746}
Committed: https://chromium.googlesource.com/chromium/src/+/0c0f9ef081109ce6fdeee588d356238272d7b9f8

[wkorman, 2017-02-28 01:00:26]

Description was changed from

==========
Increment property tree sequence number when updating.

BUG=692842
==========

to

==========
Increment property tree sequence number when updating.

BUG=692842
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[wkorman, 2017-02-28 01:05:49]

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

[commit-bot: I haz the power, 2017-02-28 01:06:22]

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

[wkorman, 2017-02-28 01:42:26]

wkorman@chromium.org changed reviewers:
+ ajuma@chromium.org, pdr@chromium.org

[wkorman, 2017-02-28 01:42:26]

I recall seeing someone (pdr@?) comment on a code review, re: overflow concerns
for a per-frame incremented number, as wangxianzhu@ having done research to show
that it's safe. If I'm off base let's discuss.

[ajuma, 2017-02-28 15:45:36]

lgtm

[pdr., 2017-02-28 17:44:59]

LGTM here as well. Looks great!

https://codereview.chromium.org/2725513002/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp
(right):

https://codereview.chromium.org/2725513002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp:65:
static int sPropertyTreeSequenceNumber = 1;
Can you add a short comment here describing what this is and how it works?

[wkorman, 2017-02-28 20:58:47]

https://codereview.chromium.org/2725513002/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp
(right):

https://codereview.chromium.org/2725513002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.cpp:65:
static int sPropertyTreeSequenceNumber = 1;
On 2017/02/28 17:44:58, pdr. wrote:
> Can you add a short comment here describing what this is and how it works?

Done.

[wkorman, 2017-02-28 20:58:54]

The CQ bit was checked by wkorman@chromium.org

[wkorman, 2017-02-28 20:58:54]

The patchset sent to the CQ was uploaded after l-g-t-m from pdr@chromium.org,
ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/2725513002/#ps40001
(title: "Add comment.")

[commit-bot: I haz the power, 2017-02-28 20:59:25]

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-02-28 23:44:57]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1488315534464240,
"parent_rev": "8a7d571a3c9fbe2bac5d02a79848d2c188a264c0", "commit_rev":
"0c0f9ef081109ce6fdeee588d356238272d7b9f8"}

[commit-bot: I haz the power, 2017-02-28 23:45:34]

Description was changed from

==========
Increment property tree sequence number when updating.

BUG=692842
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Increment property tree sequence number when updating.

BUG=692842
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2725513002
Cr-Commit-Position: refs/heads/master@{#453746}
Committed:
https://chromium.googlesource.com/chromium/src/+/0c0f9ef081109ce6fdeee588d356...
==========

[commit-bot: I haz the power, 2017-02-28 23:45:36]

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