Record scroll_layer_id instead of scrollbar_animation_controller pointer

Record scroll_layer_id_mouse_currently_captured_ instead of captured_scrollbar_animation_controller_ pointer

This issue is caused by the mouse down can change size and make
|captured_scrollbar_animation_controller_| to be a dangling pointer. So we
should record the scroll_layer id and find the animation_controller in each
MouseUp call.

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

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

[chaopeng, 2017-02-17 17:42:47]

Description was changed from

==========
Record scroll_layer_id_mouse_currently_captured_ instead of
captured_scrollbar_animation_controller_ pointer

This issue is caused by the mouse down can change size and make
|captured_scrollbar_animation_controller_| to be a dangling pointer. So we
should record the scroll_layer id and find the animation_controller in each
MouseUp call.

BUG=693414
==========

to

==========
Record scroll_layer_id_mouse_currently_captured_ instead of
captured_scrollbar_animation_controller_ pointer

This issue is caused by the mouse down can change size and make
|captured_scrollbar_animation_controller_| to be a dangling pointer. So we
should record the scroll_layer id and find the animation_controller in each
MouseUp call.

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

[chaopeng, 2017-02-17 17:43:57]

Description was changed from

==========
Record scroll_layer_id_mouse_currently_captured_ instead of
captured_scrollbar_animation_controller_ pointer

This issue is caused by the mouse down can change size and make
|captured_scrollbar_animation_controller_| to be a dangling pointer. So we
should record the scroll_layer id and find the animation_controller in each
MouseUp call.

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

to

==========
Record scroll_layer_id_mouse_currently_captured_ instead of
captured_scrollbar_animation_controller_ pointer

This issue is caused by the mouse down can change size and make
|captured_scrollbar_animation_controller_| to be a dangling pointer. So we
should record the scroll_layer id and find the animation_controller in each
MouseUp call.

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

[chaopeng, 2017-02-17 17:44:19]

chaopeng@chromium.org changed reviewers:
+ weiliangc@chromium.org

[chaopeng, 2017-02-17 17:44:20]

PTAL. Thank you.

[chaopeng, 2017-02-17 18:20:39]

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

[commit-bot: I haz the power, 2017-02-17 18:21:14]

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-02-17 22:45:15]

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

[commit-bot: I haz the power, 2017-02-17 22:45:16]

Dry run: This issue passed the CQ dry run.

[weiliangc, 2017-02-21 16:52:32]

LGTM

[chaopeng, 2017-02-21 18:00:47]

The CQ bit was checked by chaopeng@chromium.org

[commit-bot: I haz the power, 2017-02-21 18:01: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-02-21 20:13:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487700047610980,
"parent_rev": "43c2826bfce0d245221aab0183a0e9f3f883a7f4", "commit_rev":
"ff8e6a1d5aef75e6c0a8b87fc16c11d5cb69edac"}

[commit-bot: I haz the power, 2017-02-21 20:14:09]

Description was changed from

==========
Record scroll_layer_id_mouse_currently_captured_ instead of
captured_scrollbar_animation_controller_ pointer

This issue is caused by the mouse down can change size and make
|captured_scrollbar_animation_controller_| to be a dangling pointer. So we
should record the scroll_layer id and find the animation_controller in each
MouseUp call.

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

to

==========
Record scroll_layer_id_mouse_currently_captured_ instead of
captured_scrollbar_animation_controller_ pointer

This issue is caused by the mouse down can change size and make
|captured_scrollbar_animation_controller_| to be a dangling pointer. So we
should record the scroll_layer id and find the animation_controller in each
MouseUp call.

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

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

[commit-bot: I haz the power, 2017-02-21 20:14:09]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/ff8e6a1d5aef75e6c0a8b87fc16c...