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

chrome/browser/ui/gtk/other_device_menu_views_gtk.cc

+// 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.
+
+#include "chrome/browser/ui/gtk/other_device_menu_views_gtk.h"
+
+OtherDeviceMenuViews::OtherDeviceMenuViews(ui::SimpleMenuModel* menu_model)
+    : menu_model_(menu_model) {
+}
+
+OtherDeviceMenuViews::~OtherDeviceMenuViews() {
+}
+
+// TODO(jeremycho): Implement this.
+void OtherDeviceMenuViews::ShowMenu(gfx::NativeWindow window,

[Evan Stade, 2012/10/24 22:51:34]

this class should be named OtherDeviceMenuGtk

"Views" is a framework, "View" refers to MVC. So while this is a view, the class
name doesn't need to reflect that (you could make it OtherDeviceMenuViewGtk but
I'm of the opinion that's more verbose than need be, and we don't usually do
it).

[jeremycho, 2012/10/24 23:54:41]

Done.

+                                   const gfx::Point& location) {

[Evan Stade, 2012/10/24 22:51:34]

wrong indent

[jeremycho, 2012/10/24 23:54:41]

Done.

+}
+
+// OtherDeviceMenuController ---------------------------------------------------
+
+// static
+OtherDeviceMenu* OtherDeviceMenu::Create(
+    ui::SimpleMenuModel* menu_model) {
+  return new OtherDeviceMenuViews(menu_model);
+}