gtk: Split DownloadShelfContextMenuGtk out to its own header file.

chrome/browser/ui/gtk/download/download_shelf_context_menu_gtk.h

Index: chrome/browser/ui/gtk/download/download_shelf_context_menu_gtk.h
diff --git a/chrome/browser/ui/gtk/download/download_shelf_context_menu_gtk.h b/chrome/browser/ui/gtk/download/download_shelf_context_menu_gtk.h
new file mode 100644
index 0000000000000000000000000000000000000000..78c5b4ee29eeb221e6a0a0622c9666deb1efe84c
--- /dev/null
+++ b/chrome/browser/ui/gtk/download/download_shelf_context_menu_gtk.h
@@ -0,0 +1,49 @@
+// Copyright (c) 2011 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_GTK_DOWNLOAD_DOWNLOAD_SHELF_CONTEXT_MENU_GTK_H_
+#define CHROME_BROWSER_UI_GTK_DOWNLOAD_DOWNLOAD_SHELF_CONTEXT_MENU_GTK_H_
+#pragma once
+
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/download/download_shelf_context_menu.h"
+#include "chrome/browser/ui/gtk/menu_gtk.h"
+
+class BaseDownloadItemModel;
+class DownloadItemGtk;
+
+class DownloadShelfContextMenuGtk : public DownloadShelfContextMenu,
+                                    public MenuGtk::Delegate {
+ public:
+  // The constructor creates the menu and immediately pops it up.
+  // |model| is the download item model associated with this context menu,
+  // |widget| is the button that popped up this context menu, and |e| is
+  // the button press event that caused this menu to be created.

[asanka, 2011/05/20 21:23:52]

Can this comment be cleaned up while you are here?  I don't think the
constructor does what this says anymore.

[tfarina, 2011/05/20 22:15:47]

Removed.

+  DownloadShelfContextMenuGtk(BaseDownloadItemModel* model,
+                              DownloadItemGtk* download_item);
+  virtual ~DownloadShelfContextMenuGtk();
+
+  void Popup(GtkWidget* widget, GdkEventButton* event);
+
+ private:
+  // MenuGtk::Delegate:
+  virtual void StoppedShowing() OVERRIDE;
+  virtual GtkWidget* GetImageForCommandId(int command_id) const OVERRIDE;
+
+  // The menu we show on Popup(). We keep a pointer to it for a couple reasons:
+  //  * we don't want to have to recreate the menu every time it's popped up.
+  //  * we have to keep it in scope for longer than the duration of Popup(), or
+  //    completing the user-selected action races against the menu's
+  //    destruction.
+  scoped_ptr<MenuGtk> menu_;
+
+  // The download item that created us.
+  DownloadItemGtk* download_item_gtk_;
+
+  DISALLOW_COPY_AND_ASSIGN(DownloadShelfContextMenuGtk);
+};
+
+#endif  // CHROME_BROWSER_UI_GTK_DOWNLOAD_DOWNLOAD_SHELF_CONTEXT_MENU_GTK_H_