Remove redundant WebLocalFrame* parameter from DidFailProvisionalLoad

Remove redundant WebLocalFrame* parameter from DidFailProvisionalLoad in
WebFrameClient. Also updated WebFrameTest tests where appropriate, and
added a new ASSERT as pre-work for removing it from other methods.

BUG=361765

Review-Url: https://codereview.chromium.org/2827193002
Cr-Commit-Position: refs/heads/master@{#465907}
diff --git a/content/renderer/render_frame_impl.h b/content/renderer/render_frame_impl.h
index 4353e1a..bcb48d1 100644
--- a/content/renderer/render_frame_impl.h
+++ b/content/renderer/render_frame_impl.h
@@ -562,8 +562,7 @@
   void DidStartProvisionalLoad(blink::WebDataSource* data_source,
                                blink::WebURLRequest& request) override;
   void DidReceiveServerRedirectForProvisionalLoad() override;
-  void DidFailProvisionalLoad(blink::WebLocalFrame* frame,
-                              const blink::WebURLError& error,
+  void DidFailProvisionalLoad(const blink::WebURLError& error,
                               blink::WebHistoryCommitType commit_type) override;
   void DidCommitProvisionalLoad(
       blink::WebLocalFrame* frame,