Move ownership of the tool-bar's menu buttons into remoting.Toolbar.

Move ownership of the tool-bar's menu buttons into remoting.Toolbar.

This CL also reinstates the previous behaviour of MenuButton whereby a
second click on the button hides the menu, which is made possible by a
saner ownership model.

BUG=415410
Committed: https://crrev.com/f12832d351fc43392cdcfb4ec2418a9b53431fdf
Cr-Commit-Position: refs/heads/master@{#296284}

[Jamie, 2014-09-22 18:20:03]

jamiewalch@chromium.org changed reviewers:
+ kelvinp@chromium.org

[Jamie, 2014-09-22 18:20:04]

ptal

https://codereview.chromium.org/594503004/diff/1/remoting/webapp/window_frame.js
File remoting/webapp/window_frame.js (right):

https://codereview.chromium.org/594503004/diff/1/remoting/webapp/window_frame...
remoting/webapp/window_frame.js:61: base.debug.assert(this.optionsMenuList_ !=
null);
Unrelated typo fix. I can move this to a separate CL if you like.

[kelvinp, 2014-09-23 21:06:31]

lgtm

[Jamie, 2014-09-23 21:15:42]

The CQ bit was checked by jamiewalch@chromium.org

[commit-bot: I haz the power, 2014-09-23 21:16:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/594503004/20001

[commit-bot: I haz the power, 2014-09-23 22:29:41]

Committed patchset #2 (id:20001) as 17a16f9206a0e39b8eb79f873faa74b33cce899e

[commit-bot: I haz the power, 2014-09-23 22:30:40]

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