Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

ios/web/web_state/web_state_impl_unittest.mm

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "ios/web/web_state/web_state_impl.h"
 
 #include <stddef.h>
 
 #include <memory>
 
@@ skipping 194 matching lines @@
   scoped_refptr<net::HttpResponseHeaders> frame_headers(HeadersFromString(
       "HTTP/1.1 200 OK\r\n"
       "Content-Type: application/pdf\r\n"
       "Content-Language: fr\r\n"
       "X-Should-Not-Be-Here: oops\r\n"
       "\r\n"));
   // Simulate a load of a page with a frame.
   web_state_->OnHttpResponseHeadersReceived(real_headers.get(), real_url);
   web_state_->OnHttpResponseHeadersReceived(frame_headers.get(), frame_url);
   // Include a hash to be sure it's handled correctly.
-  web_state_->OnNavigationCommitted(
+  web_state_->UpdateHttpResponseHeaders(
       GURL(real_url.spec() + std::string("#baz")));
 
   // Verify that the right header set was kept.
+  ASSERT_TRUE(web_state_->GetHttpResponseHeaders());
   EXPECT_TRUE(
       web_state_->GetHttpResponseHeaders()->HasHeader("X-Should-Be-Here"));
   EXPECT_FALSE(
       web_state_->GetHttpResponseHeaders()->HasHeader("X-Should-Not-Be-Here"));
 
   // And that it was parsed correctly.
   EXPECT_EQ("text/html", web_state_->GetContentsMimeType());
   EXPECT_EQ("en", web_state_->GetContentLanguageHeader());
 }
 
 TEST_F(WebStateImplTest, ResponseHeaderClearing) {
   GURL url("http://foo.com/");
   scoped_refptr<net::HttpResponseHeaders> headers(HeadersFromString(
       "HTTP/1.1 200 OK\r\n"
       "Content-Type: text/html\r\n"
       "Content-Language: en\r\n"
       "\r\n"));
   web_state_->OnHttpResponseHeadersReceived(headers.get(), url);
 
   // There should be no headers before loading.
   EXPECT_EQ(NULL, web_state_->GetHttpResponseHeaders());
 
   // There should be headers and parsed values after loading.
-  web_state_->OnNavigationCommitted(url);
+  web_state_->UpdateHttpResponseHeaders(url);
+  ASSERT_TRUE(web_state_->GetHttpResponseHeaders());
   EXPECT_TRUE(web_state_->GetHttpResponseHeaders()->HasHeader("Content-Type"));
   EXPECT_NE("", web_state_->GetContentsMimeType());
   EXPECT_NE("", web_state_->GetContentLanguageHeader());
 
   // ... but not after loading another page, nor should there be specific
   // parsed values.
-  web_state_->OnNavigationCommitted(GURL("http://elsewhere.com/"));
+  web_state_->UpdateHttpResponseHeaders(GURL("http://elsewhere.com/"));
   EXPECT_EQ(NULL, web_state_->GetHttpResponseHeaders());
   EXPECT_EQ("", web_state_->GetContentsMimeType());
   EXPECT_EQ("", web_state_->GetContentLanguageHeader());
 }
 
 // Tests forwarding to WebStateObserver callbacks.
 TEST_F(WebStateImplTest, ObserverTest) {
   std::unique_ptr<TestWebStateObserver> observer(
       new TestWebStateObserver(web_state_.get()));
   EXPECT_EQ(web_state_.get(), observer->web_state());
@@ skipping 361 matching lines @@
   // Check that a false return value is forwarded correctly.
   EXPECT_FALSE(
       web_state_->OnScriptCommandReceived(kCommand2, value_2, kUrl2, false));
   EXPECT_FALSE(is_called_1);
   EXPECT_TRUE(is_called_2);
 
   web_state_->RemoveScriptCommandCallback(kPrefix2);
 }
 
 }  // namespace web