Move isLoading() from WebFrame to WebLocalFrame

It is a concept that doesn't make sense for a remote frame (whether the load
event has fired), and all callsites are using WebLocalFrames

BUG=

Review URL: https://codereview.chromium.org/555953002

git-svn-id: svn://svn.chromium.org/blink/trunk@181689 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp

@@ -394,12 +394,6 @@ void WebRemoteFrameImpl::loadHTMLString(
     ASSERT_NOT_REACHED();
 }
 
-bool WebRemoteFrameImpl::isLoading() const
-{
-    ASSERT_NOT_REACHED();
-    return false;
-}
-
 void WebRemoteFrameImpl::stopLoading()
 {
     ASSERT_NOT_REACHED();

third_party/WebKit/Source/web/WebRemoteFrameImpl.h

@@ -86,7 +86,6 @@ public:
     virtual void loadHTMLString(
         const WebData& html, const WebURL& baseURL, const WebURL& unreachableURL,
         bool replace) OVERRIDE;
-    virtual bool isLoading() const OVERRIDE;
     virtual void stopLoading() OVERRIDE;
     virtual WebDataSource* provisionalDataSource() const OVERRIDE;
     virtual WebDataSource* dataSource() const OVERRIDE;

third_party/WebKit/public/web/WebFrame.h

@@ -371,9 +371,6 @@ public:
                                 const WebURL& unreachableURL = WebURL(),
                                 bool replace = false) = 0;
 
-    // Returns true if the current frame is busy loading content.
-    virtual bool isLoading() const = 0;
-
     // Stops any pending loads on the frame and its children.
     virtual void stopLoading() = 0;
 

third_party/WebKit/public/web/WebLocalFrame.h

@@ -41,6 +41,9 @@ public:
 
     // Navigation State -------------------------------------------------------
 
+    // Returns true if the current frame's load event has not completed.
+    virtual bool isLoading() const = 0;
+
     // Returns true if any resource load is currently in progress. Exposed
     // primarily for use in layout tests. You probably want isLoading()
     // instead.