[MD History] Move the shared menu from history list to list container.

[MD History] Move the shared menu from history list to list container.

This CL moves the shared overflow menu in the history list up to the
history list container so that it can be used by both the history list
and the grouped history list.

This refactor also prepares for shift selection in the history list.

BUG=589363
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/456c3151c67045b6454d302490d1af26bad0905a
Cr-Commit-Position: refs/heads/master@{#406788}

[calamity, 2016-07-19 04:07:41]

Description was changed from

==========
[MD History] Move the shared menu from history list to list container.

This CL moves the shared overflow menu in the history list up to the
history list container so that it can be used by both the history list
and the grouped history list.

This refactor also prepares for shift selection in the history list.

BUG=589363
==========

to

==========
[MD History] Move the shared menu from history list to list container.

This CL moves the shared overflow menu in the history list up to the
history list container so that it can be used by both the history list
and the grouped history list.

This refactor also prepares for shift selection in the history list.

BUG=589363
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[calamity, 2016-07-19 06:49:34]

calamity@chromium.org changed reviewers:
+ tsergeant@chromium.org

[tsergeant, 2016-07-19 07:12:48]

🆒

https://codereview.chromium.org/2160173002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/md_history/history_list.js (right):

https://codereview.chromium.org/2160173002/diff/20001/chrome/browser/resource...
chrome/browser/resources/md_history/history_list.js:29: 'infinite-list.scroll':
'notifyListScroll_',
Rather than firing a new event, can you just close the menu on any scroll event
that makes it up to the list container?

[calamity, 2016-07-20 05:24:38]

https://codereview.chromium.org/2160173002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/md_history/history_list.js (right):

https://codereview.chromium.org/2160173002/diff/20001/chrome/browser/resource...
chrome/browser/resources/md_history/history_list.js:29: 'infinite-list.scroll':
'notifyListScroll_',
On 2016/07/19 07:12:47, tsergeant wrote:
> Rather than firing a new event, can you just close the menu on any scroll
event
> that makes it up to the list container?

As discussed, the scroll event doesn't bubble.

[tsergeant, 2016-07-20 07:42:02]

lgtm

[calamity, 2016-07-21 05:25:05]

The CQ bit was checked by calamity@chromium.org

[commit-bot: I haz the power, 2016-07-21 05:25:21]

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-07-21 06:35:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-21 06:38:02]

Description was changed from

==========
[MD History] Move the shared menu from history list to list container.

This CL moves the shared overflow menu in the history list up to the
history list container so that it can be used by both the history list
and the grouped history list.

This refactor also prepares for shift selection in the history list.

BUG=589363
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD History] Move the shared menu from history list to list container.

This CL moves the shared overflow menu in the history list up to the
history list container so that it can be used by both the history list
and the grouped history list.

This refactor also prepares for shift selection in the history list.

BUG=589363
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/456c3151c67045b6454d302490d1af26bad0905a
Cr-Commit-Position: refs/heads/master@{#406788}
==========

[commit-bot: I haz the power, 2016-07-21 06:38:03]

Patchset 2 (id:??) landed as
https://crrev.com/456c3151c67045b6454d302490d1af26bad0905a
Cr-Commit-Position: refs/heads/master@{#406788}