Pass MouseUp to the captured scrollbar animation controller in layer tree

Pass MouseUp to the captured scrollbar animation controller in layer tree.
This issue is caused by we only pass the MouseUp to the scoll layer currently
mouse over. If we capture the scrollbar then move to another scoll layer then
the captured scrollbar will lose the MouseUp. 

In this patch we save the captured_scrollbar_animation_controller when mouse 
down so that we can call the captured_scrollbar_animation_controller MouseUp.

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

Review-Url: https://codereview.chromium.org/2683763004
Cr-Commit-Position: refs/heads/master@{#449986}
Committed: https://chromium.googlesource.com/chromium/src/+/7250e1ab8f60d4a7ad8644c11ed0576f2c4f34b1

[chaopeng, 2017-02-08 20:30:52]

Description was changed from

==========
Pass MouseUp to all the scrollbar animation controllers in layer tree

This issue is caused by we only pass the MouseUp to the scoll layer currently
mouse over. If we capture the scrollbar then move to another scoll layer then
the captured scrollbar will lose the MouseUp. In this patch we change the
MouseUp method in layer_tree_host_impl to call all
scrollbar_animation_controller MouseUp.

BUG=690052
==========

to

==========
Pass MouseUp to all the scrollbar animation controllers in layer tree

This issue is caused by we only pass the MouseUp to the scoll layer currently
mouse over. If we capture the scrollbar then move to another scoll layer then
the captured scrollbar will lose the MouseUp. In this patch we change the
MouseUp method in layer_tree_host_impl to call all
scrollbar_animation_controller MouseUp.

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

[chaopeng, 2017-02-08 20:31:37]

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

[chaopeng, 2017-02-08 20:31:37]

PTAL. Thank you.

[bokan, 2017-02-10 15:56:41]

https://codereview.chromium.org/2683763004/diff/1/cc/trees/layer_tree_host_im...
File cc/trees/layer_tree_host_impl.cc (right):

https://codereview.chromium.org/2683763004/diff/1/cc/trees/layer_tree_host_im...
cc/trees/layer_tree_host_impl.cc:3243: for (auto& pair :
scrollbar_animation_controllers_)
This only needs to notify the one captured scrollbar_animation_controller. Lets
have MouseDown tell the LTHI (via ScrollbarAnimationControllerClient) when a
scrollbar controller is captured. Then MouseUp here can just call DidMouseUp on
the captured controller.

[chaopeng, 2017-02-10 16:11:52]

On 2017/02/10 15:56:41, bokan wrote:
>
https://codereview.chromium.org/2683763004/diff/1/cc/trees/layer_tree_host_im...
> File cc/trees/layer_tree_host_impl.cc (right):
> 
>
https://codereview.chromium.org/2683763004/diff/1/cc/trees/layer_tree_host_im...
> cc/trees/layer_tree_host_impl.cc:3243: for (auto& pair :
> scrollbar_animation_controllers_)
> This only needs to notify the one captured scrollbar_animation_controller.
Lets
> have MouseDown tell the LTHI (via ScrollbarAnimationControllerClient) when a
> scrollbar controller is captured. Then MouseUp here can just call DidMouseUp
on
> the captured controller.

Add captured_animation_controller_ for MouseUp. PTAL. Thank you.

[chaopeng, 2017-02-10 16:14:14]

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

[chaopeng, 2017-02-10 16:14:14]

weiliangc@chromium.org: PTAL, Thank you.

[bokan, 2017-02-10 16:53:48]

lgtm

https://codereview.chromium.org/2683763004/diff/20001/cc/trees/layer_tree_hos...
File cc/trees/layer_tree_host_impl.h (right):

https://codereview.chromium.org/2683763004/diff/20001/cc/trees/layer_tree_hos...
cc/trees/layer_tree_host_impl.h:797: ScrollbarAnimationController*
captured_animation_controller_;
captured_scrollbar_animation_controller_

[chaopeng, 2017-02-10 18:03:45]

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

[commit-bot: I haz the power, 2017-02-10 18:04:29]

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-10 18:48:29]

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

[commit-bot: I haz the power, 2017-02-10 18:48:30]

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

[chaopeng, 2017-02-10 19:59:02]

Patchset #4 (id:60001) has been deleted

[chaopeng, 2017-02-10 20:00:52]

Patchset #3 (id:40001) has been deleted

[chaopeng, 2017-02-10 20:00:57]

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

[commit-bot: I haz the power, 2017-02-10 20:01:52]

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-10 22:01:25]

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

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

Dry run: This issue passed the CQ dry run.

[chaopeng, 2017-02-13 15:54:42]

Description was changed from

==========
Pass MouseUp to all the scrollbar animation controllers in layer tree

This issue is caused by we only pass the MouseUp to the scoll layer currently
mouse over. If we capture the scrollbar then move to another scoll layer then
the captured scrollbar will lose the MouseUp. In this patch we change the
MouseUp method in layer_tree_host_impl to call all
scrollbar_animation_controller MouseUp.

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

to

==========
Pass MouseUp to the captured scrollbar animation controller in layer tree.
This issue is caused by we only pass the MouseUp to the scoll layer currently
mouse over. If we capture the scrollbar then move to another scoll layer then
the captured scrollbar will lose the MouseUp. 

In this patch we save the captured_scrollbar_animation_controller when mouse 
down so that we can call the captured_scrollbar_animation_controller MouseUp.

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

[weiliangc, 2017-02-13 16:20:30]

lgtm

[chaopeng, 2017-02-13 16:22:36]

The CQ bit was checked by chaopeng@chromium.org

[chaopeng, 2017-02-13 16:22:37]

The patchset sent to the CQ was uploaded after l-g-t-m from bokan@chromium.org
Link to the patchset: https://codereview.chromium.org/2683763004/#ps80001
(title: "bokan's comment addressed")

[commit-bot: I haz the power, 2017-02-13 16:22:59]

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-13 17:14:48]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1487002956715400,
"parent_rev": "c2e7e0a2c9fb07ddc5cb71ef91ac0c157330fb61", "commit_rev":
"7250e1ab8f60d4a7ad8644c11ed0576f2c4f34b1"}

[commit-bot: I haz the power, 2017-02-13 17:15:21]

Description was changed from

==========
Pass MouseUp to the captured scrollbar animation controller in layer tree.
This issue is caused by we only pass the MouseUp to the scoll layer currently
mouse over. If we capture the scrollbar then move to another scoll layer then
the captured scrollbar will lose the MouseUp. 

In this patch we save the captured_scrollbar_animation_controller when mouse 
down so that we can call the captured_scrollbar_animation_controller MouseUp.

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

to

==========
Pass MouseUp to the captured scrollbar animation controller in layer tree.
This issue is caused by we only pass the MouseUp to the scoll layer currently
mouse over. If we capture the scrollbar then move to another scoll layer then
the captured scrollbar will lose the MouseUp. 

In this patch we save the captured_scrollbar_animation_controller when mouse 
down so that we can call the captured_scrollbar_animation_controller MouseUp.

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

Review-Url: https://codereview.chromium.org/2683763004
Cr-Commit-Position: refs/heads/master@{#449986}
Committed:
https://chromium.googlesource.com/chromium/src/+/7250e1ab8f60d4a7ad8644c11ed0...
==========

[commit-bot: I haz the power, 2017-02-13 17:15:22]

Committed patchset #3 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/7250e1ab8f60d4a7ad8644c11ed0...