Cleanup FrameLoader::opener() to return a Frame*.

Source/core/loader/FrameLoader.h

Index: Source/core/loader/FrameLoader.h
diff --git a/Source/core/loader/FrameLoader.h b/Source/core/loader/FrameLoader.h
index 9d3fd82605a82b8db95c06232686b8e5173f4413..5b85ddde8ed6d2fda1115d62aece6696bc29e2dd 100644
--- a/Source/core/loader/FrameLoader.h
+++ b/Source/core/loader/FrameLoader.h
@@ -48,7 +48,6 @@
 
 namespace blink {
 
-class Resource;
 class Chrome;
 class DOMWrapperWorld;
 class DocumentLoader;
@@ -56,11 +55,13 @@ class Event;
 class FetchContext;
 class FormState;
 class FormSubmission;
+class Frame;
 class FrameLoaderClient;
 class IconController;
 class NavigationAction;
 class Page;
 class ProgressTracker;
+class Resource;

[Nate Chapin, 2014/08/15 18:59:17]

Not your fault, but is this forward declare actually needed?

[dcheng, 2014/08/16 19:56:25]

It does not appear to be needed. Removed.

 class ResourceError;
 class ResourceResponse;
 class SecurityOrigin;
@@ -155,7 +156,7 @@ public:
     void forceSandboxFlags(SandboxFlags flags) { m_forcedSandboxFlags |= flags; }
     SandboxFlags effectiveSandboxFlags() const;
 
-    LocalFrame* opener();
+    Frame* opener();
     void setOpener(LocalFrame*);
 
     void detachFromParent();