NTP5: Starting implementation of a native menu for showing other device sessions.

chrome/browser/ui/search/other_device_menu.h

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_SEARCH_OTHER_DEVICE_MENU_H_
+#define CHROME_BROWSER_UI_SEARCH_OTHER_DEVICE_MENU_H_
+
+#include "base/memory/linked_ptr.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/values.h"
+#include "ui/base/models/simple_menu_model.h"
+#include "ui/gfx/point.h"

[Dan Beam, 2012/10/12 01:10:57]

nit: can you not forward this class because it's instantiated as location_?

[jeremycho, 2012/10/12 03:51:08]

Correct.

+
+namespace content {
+class WebUI;
+}
+
+namespace views {
+class MenuRunner;
+}
+
+// A menu model that builds the contents of the other device menu.  This menu
+// is displayed when a device on the ntp_search device page is clicked and
+// contains the session for that device.  This is implemented natively rather
+// than in JavaScript to allow the menu to extend into the natively-rendered
+// section of the NTP.
+class OtherDeviceMenu : public ui::SimpleMenuModel::Delegate {
+ public:
+  OtherDeviceMenu(content::WebUI* web_ui,
+                  const std::string& session_id,
+                  const gfx::Point& location);
+  virtual ~OtherDeviceMenu();
+
+  // Displays the menu.
+  void ShowMenu();
+
+  // ui::SimpleMenuModel::Delegate overrides:

[Dan Beam, 2012/10/12 01:10:57]

nit: make these private until somebody needs to inherit from this class

[jeremycho, 2012/10/12 03:51:08]

Done.

+  virtual bool IsCommandIdChecked(int command_id) const OVERRIDE;
+  virtual bool IsCommandIdEnabled(int command_id) const OVERRIDE;
+  virtual void ExecuteCommand(int command_id) OVERRIDE;
+  virtual void ExecuteCommand(int command_id, int event_flags) OVERRIDE;
+  virtual bool GetAcceleratorForCommandId(
+      int command_id,
+      ui::Accelerator* accelerator) OVERRIDE;
+
+ private:
+  typedef std::vector<linked_ptr<DictionaryValue> > TabData;
+
+  // Adds the tabs of the |session_id_| session to the menu.
+  void AddDeviceTabs();
+

[Dan Beam, 2012/10/12 01:10:57]

nit: I think you could describe some of these a little more

[jeremycho, 2012/10/12 03:51:08]

Done.

+  content::WebUI* web_ui_;  // weak
+  std::string session_id_;
+  // The anchor point for the top-left corner of the menu.
+  gfx::Point location_;
+  ui::SimpleMenuModel menu_model_;
+  scoped_ptr<views::MenuRunner> menu_runner_;
+
+  // The tab data for each menu item, in the order in which they're displayed.
+  // |command_id| i corresponds to index i.
+  TabData tab_data_;
+
+  DISALLOW_COPY_AND_ASSIGN(OtherDeviceMenu);
+};
+
+#endif  // CHROME_BROWSER_UI_SEARCH_OTHER_DEVICE_MENU_H_