Remove FrameLoader's checkLoadComplete and checkLoadCompleteForThisFrame

Source/core/frame/Frame.h

Index: Source/core/frame/Frame.h
diff --git a/Source/core/frame/Frame.h b/Source/core/frame/Frame.h
index 8e587c593a3d47ee74572fb477c064f8f936c0a1..4a8be87fa4c3aab222d00119056a889b39ad5863 100644
--- a/Source/core/frame/Frame.h
+++ b/Source/core/frame/Frame.h
@@ -108,14 +108,13 @@ public:
     // method.
     bool isRemoteFrameTemporary() const { return m_remotePlatformLayer; }
 
-    virtual bool checkLoadComplete() = 0;
-
     // isLoading() is true when the embedder should think a load is in progress.
     // In the case of LocalFrames, it means that the frame has sent a didStartLoading()
     // callback, but not the matching didStopLoading(). Inside blink, you probably
     // want Document::loadEventFinished() instead.
     void setIsLoading(bool isLoading) { m_isLoading = isLoading; }
     bool isLoading() const { return m_isLoading; }
+    virtual bool isLoadingAsChild() const { return isLoading(); }

[dcheng, 2015/01/13 22:18:55]

Hmm. I think in the long-term, we should hide these as protected/private
members, and use some variation of
http://en.wikibooks.org/wiki/More_C++_Idioms/Friendship_and_the_Attorney-Client
to expose them. This is OK for now though.

 
 protected:
     Frame(FrameClient*, FrameHost*, FrameOwner*);