Replaced webPageOrderedClose with WebStateDelegate API.

ios/web/public/test/fakes/test_web_state_delegate.h

Index: ios/web/public/test/fakes/test_web_state_delegate.h
diff --git a/ios/web/public/test/fakes/test_web_state_delegate.h b/ios/web/public/test/fakes/test_web_state_delegate.h
index 3dfca1e4d1bec55b3f5b2894833d6d80e65739bc..549ec9cd9be625e4f26bd6d6b1c1668f1cc4069d 100644
--- a/ios/web/public/test/fakes/test_web_state_delegate.h
+++ b/ios/web/public/test/fakes/test_web_state_delegate.h
@@ -24,6 +24,11 @@ struct TestCreateNewWebStateRequest {
   bool initiated_by_user = false;
 };
 
+// Encapsulates parameters passed to CloseWebState.
+struct TestCloseWebStateRequest {
+  WebState* web_state = nullptr;

[rohitrao (ping after 24h), 2017/03/13 12:30:02]

It looks like we sometimes destroy the WebState when CloseWebState() is called,
so this may be a dangling pointer?  Is it worth making this a void* or
something, to make sure no one actually uses it?  Or should it keep the WebState
alive?

[Eugene But (OOO till 7-30), 2017/03/14 01:20:23]

We can't compare void* with WebState*, so that's not going to work for
WebStateImplTest::DelegateTest. Keeping it alive sounds like a reasonable thing
to do.

+};
+
 // Encapsulates parameters passed to OpenURLFromWebState.
 struct TestOpenURLRequest {
   TestOpenURLRequest();
@@ -72,6 +77,7 @@ class TestWebStateDelegate : public WebStateDelegate {
                               const GURL& url,
                               const GURL& opener_url,
                               bool initiated_by_user) override;
+  void CloseWebState(WebState* source) override;
   WebState* OpenURLFromWebState(WebState*,
                                 const WebState::OpenURLParams&) override;
   JavaScriptDialogPresenter* GetJavaScriptDialogPresenter(WebState*) override;
@@ -109,6 +115,11 @@ class TestWebStateDelegate : public WebStateDelegate {
     return last_create_new_web_state_request_.get();
   }
 
+  // Returns the last Web State closing request passed to |CloseWebState|.
+  TestCloseWebStateRequest* last_close_web_state_request() const {
+    return last_close_web_state_request_.get();
+  }
+
   // Returns the last Open URL request passed to |OpenURLFromWebState|.
   TestOpenURLRequest* last_open_url_request() const {
     return last_open_url_request_.get();
@@ -144,6 +155,7 @@ class TestWebStateDelegate : public WebStateDelegate {
   bool handle_context_menu_called_ = false;
   std::unique_ptr<TestCreateNewWebStateRequest>
       last_create_new_web_state_request_;
+  std::unique_ptr<TestCloseWebStateRequest> last_close_web_state_request_;
   std::unique_ptr<TestOpenURLRequest> last_open_url_request_;
   std::unique_ptr<TestRepostFormRequest> last_repost_form_request_;
   bool get_java_script_dialog_presenter_called_ = false;