Do not allow URL drops on app windows to cause a navigation away

from the currently loaded site.  We are careful in this patch to
continue to allow dropping URLs in text fields within the app
window, and behavior for normal browser windows remains as
before.

There is a slight glitch when dragging a to an app window on the
border of the window.  Even though it is very brief, it is still
disturbing.

BUG=7171
TEST=Open Chrome (1), load google.com.  Open Chrome (2), load
yahoo.com.  Drag a link from 1 to 2 and a link from 2 to 1 (both
allowed).  Create an app shortcut from 1, drag a link from 1 to
2 (allowed) and a link from 2 to 1 (denied).  Verify that link
scan be dragged to the omnibox and to text fields.

Patch by Chase Phillips <chase@chromium.org> via
http://codereview.chromium.org/119298

Review URL: http://codereview.chromium.org/121003


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18100 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 7 insertions, 0 deletions

diff --git a/chrome/renderer/render_view.h b/chrome/renderer/render_view.h
index a0dd47a..f661b02 100644
--- a/chrome/renderer/render_view.h
+++ b/chrome/renderer/render_view.h
@@ -18,6 +18,7 @@
 #include "base/timer.h"
 #include "base/values.h"
 #include "build/build_config.h"
+#include "chrome/browser/renderer_preferences.h"
 #include "chrome/renderer/automation/dom_automation_controller.h"
 #include "chrome/renderer/dom_ui_bindings.h"
 #include "chrome/renderer/extensions/extension_process_bindings.h"
@@ -113,6 +114,7 @@ class RenderView : public RenderWidget,
       gfx::NativeViewId parent_hwnd,
       base::WaitableEvent* modal_dialog_event,  // takes ownership
       int32 opener_id,
+      const RendererPreferences& renderer_prefs,
       const WebPreferences& webkit_prefs,
       SharedRenderViewCounter* counter,
       int32 routing_id);
@@ -137,6 +139,7 @@ class RenderView : public RenderWidget,
   virtual void OnMessageReceived(const IPC::Message& msg);
 
   // WebViewDelegate
+  virtual bool CanAcceptLoadDrops() const;
   virtual void ShowModalHTMLDialog(const GURL& url, int width, int height,
                                    const std::string& json_arguments,
                                    std::string* json_retval);
@@ -410,6 +413,7 @@ class RenderView : public RenderWidget,
   void Init(gfx::NativeViewId parent,
             base::WaitableEvent* modal_dialog_event,  // takes ownership
             int32 opener_id,
+            const RendererPreferences& renderer_prefs,
             const WebPreferences& webkit_prefs,
             SharedRenderViewCounter* counter,
             int32 routing_id);
@@ -540,6 +544,7 @@ class RenderView : public RenderWidget,
   void OnFileChooserResponse(const std::vector<FilePath>& file_names);
   void OnEnableViewSourceMode();
   void OnEnableIntrinsicWidthChangedMode();
+  void OnSetRendererPrefs(const RendererPreferences& renderer_prefs);
   void OnUpdateBackForwardListCount(int back_list_count,
                                     int forward_list_count);
   void OnGetAccessibilityInfo(
@@ -799,6 +804,8 @@ class RenderView : public RenderWidget,
   // Need for printing
   scoped_ptr<PrintWebViewHelper> print_render_view_;
 
+  RendererPreferences renderer_preferences_;
+
   DISALLOW_COPY_AND_ASSIGN(RenderView);
 };