Conversion of content_shell target to use WebLocalFrame.

content/public/renderer/render_frame.h

Index: content/public/renderer/render_frame.h
diff --git a/content/public/renderer/render_frame.h b/content/public/renderer/render_frame.h
index f7de4467c1f6ecf18f79ae98baadf3ac6db67aec..19feb7d557faf6a50f97a06612bd583d1c7949c1 100644
--- a/content/public/renderer/render_frame.h
+++ b/content/public/renderer/render_frame.h
@@ -9,6 +9,8 @@
 #include "content/common/content_export.h"
 #include "ipc/ipc_listener.h"
 #include "ipc/ipc_sender.h"
+// TODO(dcheng): Convert back to a forward declare.
+#include "third_party/WebKit/public/web/WebLocalFrame.h"

[Charlie Reis, 2014/04/02 23:34:15]

Why is this needed for now?  And what will change to allow us to remove it
later?

[dcheng, 2014/04/02 23:47:08]

WebLocalFrame is currently a typedef, and you cannot forward declare typedefs.
However, I cannot make it a class either--the alternative was to temporarily
double up all the methods in public/web that are changing to take a
WebLocalFrame.

Once this conversion is complete, I will convert WebLocalFrame to be a subclass
so it can be forward declared in most places again.

[Charlie Reis, 2014/04/03 00:37:11]

I see, thanks.

 #include "third_party/WebKit/public/web/WebNavigationPolicy.h"
 
 struct WebPreferences;
@@ -77,10 +79,9 @@ class CONTENT_EXPORT RenderFrame : public IPC::Listener,
       const blink::WebPluginParams& params) = 0;
 
   // The client should handle the navigation externally.
-  virtual void LoadURLExternally(
-      blink::WebFrame* frame,
-      const blink::WebURLRequest& request,
-      blink::WebNavigationPolicy policy) = 0;
+  virtual void LoadURLExternally(blink::WebLocalFrame* frame,
+                                 const blink::WebURLRequest& request,
+                                 blink::WebNavigationPolicy policy) = 0;
 
   // Execute a string of JavaScript in this frame's context.
   virtual void ExecuteJavaScript(const base::string16& javascript) = 0;