Fix dragging supported files to tab strip.

chrome/browser/ui/views/tabs/tab_strip.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_TABS_TAB_STRIP_H_
 #define CHROME_BROWSER_UI_VIEWS_TABS_TAB_STRIP_H_
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 373 matching lines @@
   // it was reverted.
   void StoppedDraggingTabs(const std::vector<Tab*>& tabs,
                            const std::vector<int>& initial_positions,
                            bool move_only,
                            bool completed);
 
   // Invoked from StoppedDraggingTabs to cleanup |tab|. If |tab| is known
   // |is_first_tab| is set to true.
   void StoppedDraggingTab(Tab* tab, bool* is_first_tab);
 
+  // Called when a MIME type request for a dragged file has been completed.
+  void OnFindURLMimeTypeCompleted(GURL url, const std::string& mime_type);
+
   // Takes ownership of |controller|.
   void OwnDragController(TabDragController* controller);
 
   // Destroys the current TabDragController. This cancel the existing drag
   // operation.
   void DestroyDragController();
 
   // Releases ownership of the current TabDragController.
   TabDragController* ReleaseDragController();
 
@@ skipping 168 matching lines @@
   // cursor.
   int available_width_for_tabs_;
 
   // True if PrepareForCloseAt has been invoked. When true remove animations
   // preserve current tab bounds.
   bool in_tab_close_;
 
   // Valid for the lifetime of a drag over us.
   scoped_ptr<DropInfo> drop_info_;
 
+  // The URL from the latest DropTargetEvent.

[sky, 2013/11/15 00:48:54]

All of this state, including the weak factory, should move to DropInfo. You may
need to change the life of DropInfo, but that's ok.

[michaelpg, 2013/11/18 23:24:51]

I've moved the URL and supported boolean into DropInfo. I moved the
WeakFactoryPtr to browser_tab_strip_controller, since that's where the MIME type
will call back so we can check for profile plugins, if that makes sense.

+  GURL drag_url_;
+
+  // Whether the MIME type of the file pointed to by drag_url_ is supported.
+  // Defaults to true.
+  bool drag_file_supported_;
+
   // To ensure all tabs pulse at the same time they share the same animation
   // container. This is that animation container.
   scoped_refptr<gfx::AnimationContainer> animation_container_;
 
   // MouseWatcher is used for two things:
   // . When a tab is closed to reset the layout.
   // . When a mouse is used and the layout dynamically adjusts and is currently
   //   TAB_STRIP_LAYOUT_STACKED.
   scoped_ptr<views::MouseWatcher> mouse_watcher_;
 
@@ skipping 31 matching lines @@
   // tab close comes from a touch device.
   base::OneShotTimer<TabStrip> resize_layout_timer_;
 
   // True if tabs are painted as rectangular light-bars.
   bool immersive_style_;
 
   // Our observers.
   typedef ObserverList<TabStripObserver> TabStripObservers;
   TabStripObservers observers_;
 
+  base::WeakPtrFactory<TabStrip> weak_ptr_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(TabStrip);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TABS_TAB_STRIP_H_