Prevented connecting drag drop events to a SwappedOut RenderViewHost in WebContentsViewGtk

content/browser/web_contents/web_contents_view_gtk.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 CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_VIEW_GTK_H_
 #define CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_VIEW_GTK_H_
 
 #include <gtk/gtk.h>
 
 #include <vector>
@@ skipping 79 matching lines @@
   virtual void GotFocus() OVERRIDE;
   virtual void TakeFocus(bool reverse) OVERRIDE;
 
  private:
   // Insert the given widget into the content area. Should only be used for
   // web pages and the like (including interstitials and sad tab). Note that
   // this will be perfectly happy to insert overlapping render views, so care
   // should be taken that the correct one is hidden/shown.
   void InsertIntoContentArea(GtkWidget* widget);
 
+  void UpdateDragDest(RenderViewHost* new_host);

[Charlie Reis, 2013/03/14 22:08:38]

Please add a comment saying when this needs to be called (i.e., when a new
RenderViewHost swap in).

[mthiesse, 2013/03/15 15:26:49]

Done.

+
   // Handle focus traversal on the render widget native view. Can be overridden
   // by subclasses.
   CHROMEGTK_CALLBACK_1(WebContentsViewGtk, gboolean, OnFocus, GtkDirectionType);
 
   // Used to adjust the size of its children when the size of |expanded_| is
   // changed.
   CHROMEGTK_CALLBACK_2(WebContentsViewGtk, void, OnChildSizeRequest,
                        GtkWidget*, GtkRequisition*);
 
   // Used to propagate the size change of |expanded_| to our RWHV to resize the
@@ skipping 25 matching lines @@
   // The size we want the view to be.  We keep this in a separate variable
   // because resizing in GTK+ is async.
   gfx::Size requested_size_;
 
   DISALLOW_COPY_AND_ASSIGN(WebContentsViewGtk);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_VIEW_GTK_H_