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

Unified Diff: chrome/browser/tab_contents/web_drag_bookmark_handler_mac.mm

Issue 8430013: Split Mac WebDropTarget into two, content and chrome. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: removed commented out code Created 9 years, 2 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_mac.mm
diff --git a/chrome/browser/tab_contents/web_drag_bookmark_handler_mac.mm b/chrome/browser/tab_contents/web_drag_bookmark_handler_mac.mm
new file mode 100644
index 0000000000000000000000000000000000000000..ad1b37536f23ad64dba3445e4c85baf074ec551e
--- /dev/null
+++ b/chrome/browser/tab_contents/web_drag_bookmark_handler_mac.mm
@@ -0,0 +1,58 @@
+// 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.
+
+#include "chrome/browser/tab_contents/web_drag_bookmark_handler_mac.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"
+
+WebDragBookmarkHandlerMac::WebDragBookmarkHandlerMac()
+ : tab_(NULL) {
+}
+
+WebDragBookmarkHandlerMac::~WebDragBookmarkHandlerMac() {}
+
+void WebDragBookmarkHandlerMac::DragInitialize(TabContents* contents) {
jochen (gone - plz use gerrit) 2011/11/01 15:16:07 maybe DCHECK that contents and tab are the same if
Avi (use Gerrit) 2011/11/01 16:01:06 Done.
+ if (!tab_)
+ tab_ = TabContentsWrapper::GetCurrentWrapperForContents(contents);
+
+ bookmark_drag_data_.ReadFromDragClipboard();
+}
+
+void WebDragBookmarkHandlerMac::OnDragOver() {
+ if (tab_ && tab_->bookmark_tab_helper()->GetBookmarkDragDelegate())
jochen (gone - plz use gerrit) 2011/11/01 15:16:07 brackets since if-body >= 1 line
Avi (use Gerrit) 2011/11/01 16:01:06 will fix in source too
+ tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()->OnDragOver(
+ bookmark_drag_data_);
+}
+
+void WebDragBookmarkHandlerMac::OnDragEnter() {
+ if (tab_ && tab_->bookmark_tab_helper()->GetBookmarkDragDelegate())
jochen (gone - plz use gerrit) 2011/11/01 15:16:07 same here
+ tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()->OnDragEnter(
+ bookmark_drag_data_);
+}
+
+void WebDragBookmarkHandlerMac::OnDrop() {
+ // This is non-null if tab_contents_ is showing an ExtensionWebUI with
+ // support for (at the moment experimental) drag and drop extensions.
+ if (tab_) {
+ if (tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()) {
+ tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()->OnDrop(
+ bookmark_drag_data_);
+ }
+
+ // Focus the target browser.
+ Browser* browser = Browser::GetBrowserForController(
+ &tab_->controller(), NULL);
+ if (browser)
+ browser->window()->Show();
+ }
+}
+
+void WebDragBookmarkHandlerMac::OnDragLeave() {
+ if (tab_ && tab_->bookmark_tab_helper()->GetBookmarkDragDelegate())
jochen (gone - plz use gerrit) 2011/11/01 15:16:07 and here
+ tab_->bookmark_tab_helper()->GetBookmarkDragDelegate()->OnDragLeave(
+ bookmark_drag_data_);
+}

Powered by Google App Engine
This is Rietveld 408576698