Implemented drop hander for the Home toolbar button that accepts (only) links

chrome/browser/ui/views/home_button.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_VIEWS_HOME_BUTTON_H__
+#define CHROME_BROWSER_UI_VIEWS_HOME_BUTTON_H__
+
+#include <set>
+
+#include "ui/views/controls/button/image_button.h"
+
+class Browser;
+
+class HomeImageButton : public views::ImageButton {
+ public:
+  HomeImageButton(views::ButtonListener* listener, Browser* browser);
+  virtual ~HomeImageButton();
+
+  virtual bool GetDropFormats(
+      int* formats,
+      std::set<OSExchangeData::CustomFormat>* custom_formats) OVERRIDE;
+  virtual bool CanDrop(const OSExchangeData& data) OVERRIDE;
+  virtual int OnDragUpdated(const ui::DropTargetEvent& event) OVERRIDE;
+  virtual int OnPerformDrop(const ui::DropTargetEvent& event) OVERRIDE;
+
+ private:
+  Browser* browser_;
+};

[Peter Kasting, 2013/01/18 18:19:27]

Nit: DISALLOW_COPY_AND_ASSIGN()

[Tom Cassiotis, 2013/01/18 21:24:19]

Done.

+
+#endif  // CHROME_BROWSER_UI_VIEWS_HOME_BUTTON_H__
+

[Peter Kasting, 2013/01/18 18:19:27]

Extra newline.  I know you added this because Rietveld's lint complained before.
 Ignore it.  As long as your local file has an LF as the last character, you're
fine.  Lint often spits out warnings about this that are totally wrong.

(2 files)

[Tom Cassiotis, 2013/01/18 21:24:19]

Done.