Add a new ContentRendererClient::HandleNavigation callback in RenderViewImpl::decidePolicyForNaviga…

content/public/renderer/content_renderer_client.h

Index: content/public/renderer/content_renderer_client.h
===================================================================
--- content/public/renderer/content_renderer_client.h	(revision 165669)
+++ content/public/renderer/content_renderer_client.h	(working copy)
@@ -12,6 +12,8 @@
 #include "ipc/ipc_message.h"
 #include "content/public/common/content_client.h"
 #include "content/public/common/page_transition_types.h"
+#include "third_party/WebKit/Source/WebKit/chromium/public/WebNavigationPolicy.h"
+#include "third_party/WebKit/Source/WebKit/chromium/public/WebNavigationType.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebPageVisibilityState.h"
 #include "v8/include/v8.h"
 
@@ -130,6 +132,14 @@
   // Returns true if the given url can create popup windows.
   virtual bool AllowPopup(const GURL& creator);
 
+  // Returns true if the navigation was handled by the embedder and should be
+  // ignored by WebKit.
+  virtual bool HandleNavigation(WebKit::WebFrame* frame,

[darin (slow to review), 2012/11/09 00:30:24]

One question:  How do Chromium developers know not to delete this function?  "It
looks like no one uses this."

[jam, 2012/11/09 00:32:37]

hmm, good question. how do you handle this for WebKit API? or is every function
on it solely for chromium?

+                                const WebKit::WebURLRequest& request,
+                                WebKit::WebNavigationType type,
+                                WebKit::WebNavigationPolicy default_policy,
+                                bool is_redirect);
+
   // Returns true if we should fork a new process for the given navigation.
   virtual bool ShouldFork(WebKit::WebFrame* frame,
                           const GURL& url,