Moving hiding of paste menu from scroll start to scroll update

Moving hiding of paste menu from scroll start to scroll update

Previously we were hiding the paste menu even when the page didn't
scroll at all. Moving the hiding to update ensures that the menu
is only cleared if we actually scrolled.

BUG=673209
Committed: https://crrev.com/99f54b1704faa5413d106b23a64a434368ea9628
Cr-Commit-Position: refs/heads/master@{#438708}

[amaralp, 2016-12-14 23:30:50]

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

[amaralp, 2016-12-14 23:31:05]

amaralp@chromium.org changed reviewers:
+ aelias@chromium.org

[amaralp, 2016-12-14 23:31:19]

Description was changed from

==========
Moving hiding of paste menu from scroll start to scroll update

Previously we were clearing the paste menu even when the page didn't
scroll at all. Moving the hiding to update ensures that the menu
is only cleared if we actually scrolled.

BUG=673209
==========

to

==========
Moving hiding of paste menu from scroll start to scroll update

Previously we were hiding the paste menu even when the page didn't
scroll at all. Moving the hiding to update ensures that the menu
is only cleared if we actually scrolled.

BUG=673209
==========

[amaralp, 2016-12-14 23:31:37]

PTAL

[commit-bot: I haz the power, 2016-12-14 23:32:35]

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

[aelias_OOO_until_Jul13, 2016-12-14 23:45:25]

lgtm

[commit-bot: I haz the power, 2016-12-15 00:11:30]

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

[commit-bot: I haz the power, 2016-12-15 00:11:32]

Dry run: This issue passed the CQ dry run.

[amaralp, 2016-12-15 01:01:45]

The CQ bit was checked by amaralp@chromium.org

[commit-bot: I haz the power, 2016-12-15 01:03:09]

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-12-15 01:35:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481763705878400, "parent_rev":
"a4773786aef9141a1a10c9a3d39a7a3bd71384c7", "commit_rev":
"3de67997af5db61e02b5018911d55bcc7a9f23aa"}

[commit-bot: I haz the power, 2016-12-15 01:35:30]

Description was changed from

==========
Moving hiding of paste menu from scroll start to scroll update

Previously we were hiding the paste menu even when the page didn't
scroll at all. Moving the hiding to update ensures that the menu
is only cleared if we actually scrolled.

BUG=673209
==========

to

==========
Moving hiding of paste menu from scroll start to scroll update

Previously we were hiding the paste menu even when the page didn't
scroll at all. Moving the hiding to update ensures that the menu
is only cleared if we actually scrolled.

BUG=673209

Review-Url: https://codereview.chromium.org/2580583002
==========

[commit-bot: I haz the power, 2016-12-15 01:35:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-15 01:38:53]

Description was changed from

==========
Moving hiding of paste menu from scroll start to scroll update

Previously we were hiding the paste menu even when the page didn't
scroll at all. Moving the hiding to update ensures that the menu
is only cleared if we actually scrolled.

BUG=673209

Review-Url: https://codereview.chromium.org/2580583002
==========

to

==========
Moving hiding of paste menu from scroll start to scroll update

Previously we were hiding the paste menu even when the page didn't
scroll at all. Moving the hiding to update ensures that the menu
is only cleared if we actually scrolled.

BUG=673209

Committed: https://crrev.com/99f54b1704faa5413d106b23a64a434368ea9628
Cr-Commit-Position: refs/heads/master@{#438708}
==========

[commit-bot: I haz the power, 2016-12-15 01:38:54]

Patchset 1 (id:??) landed as
https://crrev.com/99f54b1704faa5413d106b23a64a434368ea9628
Cr-Commit-Position: refs/heads/master@{#438708}