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

Source/core/loader/FrameLoader.h

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) Research In Motion Limited 2009. All rights reserved.
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
@@ skipping 30 matching lines @@
 #include "core/loader/HistoryItem.h"
 #include "core/loader/MixedContentChecker.h"
 #include "platform/Timer.h"
 #include "platform/network/ResourceRequest.h"
 #include "wtf/Forward.h"
 #include "wtf/HashSet.h"
 #include "wtf/OwnPtr.h"
 
 namespace blink {
 
-class Resource;
 class Chrome;
 class DOMWrapperWorld;
 class DocumentLoader;
 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;
 class SerializedScriptValue;
 class SubstituteData;
 
 struct FrameLoadRequest;
 struct WindowFeatures;
 
 bool isBackForwardLoadType(FrameLoadType);
@@ skipping 74 matching lines @@
 
     void dispatchDidClearWindowObjectInMainWorld();
     void dispatchDidClearDocumentOfWindowObject();
     void dispatchDocumentElementAvailable();
 
     // The following sandbox flags will be forced, regardless of changes to
     // the sandbox attribute of any parent frames.
     void forceSandboxFlags(SandboxFlags flags) { m_forcedSandboxFlags |= flags; }
     SandboxFlags effectiveSandboxFlags() const;
 
-    LocalFrame* opener();
+    Frame* opener();
     void setOpener(LocalFrame*);
 
     void detachFromParent();
 
     void loadDone();
     void finishedParsing();
     void checkCompleted();
 
     void commitProvisionalLoad();
 
@@ skipping 108 matching lines @@
     Timer<FrameLoader> m_didAccessInitialDocumentTimer;
 
     SandboxFlags m_forcedSandboxFlags;
 
     bool m_willDetachClient;
 };
 
 } // namespace blink
 
 #endif // FrameLoader_h