Clarify property tree id-to-index map names.

Clarify property tree id-to-index map names.

Follow-up from discussion on http://crrev.com/2612093002.

BUG=674258
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2616353003
Cr-Commit-Position: refs/heads/master@{#443027}
Committed: https://chromium.googlesource.com/chromium/src/+/c2d1f65f2eaf620936ed56c42fe3e6235674600b

[wkorman, 2017-01-10 00:19:59]

Description was changed from

==========
Clarify property tree id-to-index map names.

Follow-up from discussion on http://crrev.com/2612093002.
==========

to

==========
Clarify property tree id-to-index map names.

Follow-up from discussion on http://crrev.com/2612093002.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[wkorman, 2017-01-10 00:21:23]

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

[wkorman, 2017-01-10 00:21:23]

Makes these more self-documenting and in line with names in
http://crrev.com/2612093002.

[ajuma, 2017-01-10 00:51:44]

lgtm

https://codereview.chromium.org/2616353003/diff/1/cc/input/scrollbar_animatio...
File cc/input/scrollbar_animation_controller_linear_fade.cc (right):

https://codereview.chromium.org/2616353003/diff/1/cc/input/scrollbar_animatio...
cc/input/scrollbar_animation_controller_linear_fade.cc:67: // are
nit: needs re-wrapping

https://codereview.chromium.org/2616353003/diff/1/cc/input/scrollbar_animatio...
File cc/input/scrollbar_animation_controller_thinning.cc (right):

https://codereview.chromium.org/2616353003/diff/1/cc/input/scrollbar_animatio...
cc/input/scrollbar_animation_controller_thinning.cc:220: // are
wrapping

[wkorman, 2017-01-10 01:13:43]

https://codereview.chromium.org/2616353003/diff/1/cc/input/scrollbar_animatio...
File cc/input/scrollbar_animation_controller_linear_fade.cc (right):

https://codereview.chromium.org/2616353003/diff/1/cc/input/scrollbar_animatio...
cc/input/scrollbar_animation_controller_linear_fade.cc:67: // are
On 2017/01/10 at 00:51:44, ajuma wrote:
> nit: needs re-wrapping

Done.

https://codereview.chromium.org/2616353003/diff/1/cc/input/scrollbar_animatio...
File cc/input/scrollbar_animation_controller_thinning.cc (right):

https://codereview.chromium.org/2616353003/diff/1/cc/input/scrollbar_animatio...
cc/input/scrollbar_animation_controller_thinning.cc:220: // are
On 2017/01/10 at 00:51:44, ajuma wrote:
> wrapping

Done.

[wkorman, 2017-01-10 01:13:53]

The CQ bit was checked by wkorman@chromium.org

[wkorman, 2017-01-10 01:13:53]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/2616353003/#ps20001
(title: "Wrap comments.")

[commit-bot: I haz the power, 2017-01-10 01:14:51]

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-01-10 01:34:46]

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

[commit-bot: I haz the power, 2017-01-10 01:34:47]

Try jobs failed on following builders:
  linux_trusty_blink_rel on master.tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_trusty_blink_rel/b...)

[wkorman, 2017-01-10 20:55:49]

The CQ bit was checked by wkorman@chromium.org

[wkorman, 2017-01-10 20:55:50]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/2616353003/#ps40001
(title: "Rename in PaintArtifactCompositor.")

[commit-bot: I haz the power, 2017-01-10 20:56:17]

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-01-10 21:04:47]

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

[commit-bot: I haz the power, 2017-01-10 21:04:48]

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

[wkorman, 2017-01-10 21:08:20]

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

[wkorman, 2017-01-10 21:08:21]

+chrishtr for Blink platform OWNERS. Just a field rename.

[chrishtr, 2017-01-10 21:15:51]

lgtm

[wkorman, 2017-01-10 21:34:25]

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

[commit-bot: I haz the power, 2017-01-10 21:34:54]

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, 2017-01-10 22:26:44]

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

[commit-bot: I haz the power, 2017-01-10 22:26:45]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[wkorman, 2017-01-10 23:25:20]

The CQ bit was checked by wkorman@chromium.org

[wkorman, 2017-01-10 23:25:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrishtr@chromium.org, ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/2616353003/#ps60001
(title: "Sync to head.")

[commit-bot: I haz the power, 2017-01-10 23:26:05]

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-01-11 00:26:33]

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

[commit-bot: I haz the power, 2017-01-11 00:26:34]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[wkorman, 2017-01-11 03:22:43]

The CQ bit was checked by wkorman@chromium.org

[commit-bot: I haz the power, 2017-01-11 03:23:38]

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

[wkorman, 2017-01-11 03:29:05]

Description was changed from

==========
Clarify property tree id-to-index map names.

Follow-up from discussion on http://crrev.com/2612093002.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Clarify property tree id-to-index map names.

Follow-up from discussion on http://crrev.com/2612093002.

BUG=674258
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[commit-bot: I haz the power, 2017-01-11 03:30:20]

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

[commit-bot: I haz the power, 2017-01-11 03:30:21]

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

[wkorman, 2017-01-11 04:05:48]

The CQ bit was checked by wkorman@chromium.org

[wkorman, 2017-01-11 04:05:48]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org,
chrishtr@chromium.org
Link to the patchset: https://codereview.chromium.org/2616353003/#ps80001
(title: "Sync to head.")

[commit-bot: I haz the power, 2017-01-11 04:06:20]

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-01-11 05:39:48]

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

[commit-bot: I haz the power, 2017-01-11 05:39:49]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[wkorman, 2017-01-11 07:27:54]

The CQ bit was checked by wkorman@chromium.org

[commit-bot: I haz the power, 2017-01-11 07:28:24]

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-01-11 08:34:28]

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

[commit-bot: I haz the power, 2017-01-11 08:34:30]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[wkorman, 2017-01-11 21:09:26]

Filed http://crbug.com/680254 to track apparent issue with the Android swarming
build, but going to try commit once again regardless.

[wkorman, 2017-01-11 21:09:34]

The CQ bit was checked by wkorman@chromium.org

[wkorman, 2017-01-11 21:09:34]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org,
chrishtr@chromium.org
Link to the patchset: https://codereview.chromium.org/2616353003/#ps100001
(title: "Sync to head.")

[commit-bot: I haz the power, 2017-01-11 21:10:08]

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-01-11 22:45:36]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1484168974171070,
"parent_rev": "f4f7c520efaa03663cd8c19662f915bf313bfe7b", "commit_rev":
"c2d1f65f2eaf620936ed56c42fe3e6235674600b"}

[commit-bot: I haz the power, 2017-01-11 22:46:13]

Description was changed from

==========
Clarify property tree id-to-index map names.

Follow-up from discussion on http://crrev.com/2612093002.

BUG=674258
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Clarify property tree id-to-index map names.

Follow-up from discussion on http://crrev.com/2612093002.

BUG=674258
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2616353003
Cr-Commit-Position: refs/heads/master@{#443027}
Committed:
https://chromium.googlesource.com/chromium/src/+/c2d1f65f2eaf620936ed56c42fe3...
==========

[commit-bot: I haz the power, 2017-01-11 22:46:14]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/c2d1f65f2eaf620936ed56c42fe3...