Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(282)

Unified Diff: chrome/browser/tab_contents/web_drag_bookmark_handler_aura.cc

Issue 9757001: Support custom drag-and-drop of bookmarks in Aura (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Attempt to fix win aura linker error Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/tab_contents/web_drag_bookmark_handler_aura.cc
diff --git a/chrome/browser/tab_contents/web_drag_bookmark_handler_aura.cc b/chrome/browser/tab_contents/web_drag_bookmark_handler_aura.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d358459bacbb6e071789101ecad4cf7af468b205
--- /dev/null
+++ b/chrome/browser/tab_contents/web_drag_bookmark_handler_aura.cc
@@ -0,0 +1,88 @@
+// 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/tab_contents/web_drag_bookmark_handler_aura.h"
+
+#include "chrome/browser/bookmarks/bookmark_node_data.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_window.h"
+#include "chrome/browser/ui/bookmarks/bookmark_tab_helper.h"
+#include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
+#include "content/public/browser/web_contents.h"
+#include "ui/base/dragdrop/os_exchange_data.h"
+#include "ui/base/dragdrop/os_exchange_data_provider_aura.h"
+#include "webkit/glue/webdropdata.h"
+
+using content::WebContents;
+
+WebDragBookmarkHandlerAura::WebDragBookmarkHandlerAura()
+ : tab_(NULL) {
+}
+
+WebDragBookmarkHandlerAura::~WebDragBookmarkHandlerAura() {
+}
+
+void WebDragBookmarkHandlerAura::DragInitialize(WebContents* contents) {
+ // Ideally we would want to initialize the the TabContentsWrapper member in
+ // the constructor. We cannot do that as the WebDragDest object is
+ // created during the construction of the TabContents object. The
+ // TabContentsWrapper is created much later.
+ DCHECK(tab_ ? (tab_->web_contents() == contents) : true);
+ if (!tab_)
+ tab_ = TabContentsWrapper::GetCurrentWrapperForContents(contents);
+}
+
+void WebDragBookmarkHandlerAura::OnDragOver() {
+ if (tab_ && tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()) {
+ if (bookmark_drag_data_.is_valid())
+ tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()->OnDragOver(
+ bookmark_drag_data_);
+ }
+}
+
+void WebDragBookmarkHandlerAura::OnReceiveDragData(
+ const ui::OSExchangeData& data) {
+ if (tab_ && tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()) {
+ // Read the bookmark drag data and save it for use in later events in this
+ // drag.
+ bookmark_drag_data_.Read(data);
+ }
+}
+
+void WebDragBookmarkHandlerAura::OnDragEnter() {
+ if (tab_ && tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()) {
+ if (bookmark_drag_data_.is_valid())
+ tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()->OnDragEnter(
+ bookmark_drag_data_);
+ }
+}
+
+void WebDragBookmarkHandlerAura::OnDrop() {
+ if (tab_) {
+ if (tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()) {
+ if (bookmark_drag_data_.is_valid()) {
+ tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()->OnDrop(
+ bookmark_drag_data_);
Ben Goodger (Google) 2012/03/22 16:19:46 4-space indent
Rick Byers 2012/03/22 16:27:41 Done.
+ }
+ }
+
+ // Focus the target browser.
+ Browser* browser = Browser::GetBrowserForController(
+ &tab_->web_contents()->GetController(), NULL);
+ if (browser)
+ browser->window()->Show();
+ }
+
+ bookmark_drag_data_.Clear();
+}
+
+void WebDragBookmarkHandlerAura::OnDragLeave() {
+ if (tab_ && tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()) {
+ if (bookmark_drag_data_.is_valid())
+ tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()->OnDragLeave(
+ bookmark_drag_data_);
Ben Goodger (Google) 2012/03/22 16:19:46 nit: 4-space idnent
Rick Byers 2012/03/22 16:27:41 Done.
+ }
+
+ bookmark_drag_data_.Clear();
+}

Powered by Google App Engine
This is Rietveld 408576698