Cleaned up unused methods from WebStateImpl and NavigationContextImpl.

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 333 matching lines @@
   SetIgnoreRenderProcessCrashesDuringTesting(true);
   ASSERT_FALSE(observer->render_process_gone_info());
   web_state_->OnRenderProcessGone();
   ASSERT_TRUE(observer->render_process_gone_info());
   EXPECT_EQ(web_state_.get(), observer->render_process_gone_info()->web_state);
 
   // Test that ProvisionalNavigationStarted() is called.
   ASSERT_FALSE(observer->did_finish_navigation_info());
   const GURL url("http://test");
   std::unique_ptr<web::NavigationContext> context =
-      NavigationContextImpl::CreateNavigationContext(
-          web_state_.get(), url, nullptr /* response_headers */);
+      NavigationContextImpl::CreateNavigationContext(web_state_.get(), url);
   web_state_->OnNavigationFinished(context.get());
   ASSERT_TRUE(observer->did_finish_navigation_info());
   EXPECT_EQ(web_state_.get(),
             observer->did_finish_navigation_info()->web_state);
   NavigationContext* actual_context =
       observer->did_finish_navigation_info()->context.get();
   EXPECT_EQ(context->GetUrl(), actual_context->GetUrl());
   EXPECT_FALSE(actual_context->IsSameDocument());
   EXPECT_FALSE(actual_context->IsErrorPage());
   EXPECT_FALSE(actual_context->GetResponseHeaders());
@@ skipping 37 matching lines @@
   ASSERT_FALSE(observer->load_page_info());
   web_state_->OnPageLoaded(url, false);
   ASSERT_TRUE(observer->load_page_info());
   EXPECT_EQ(web_state_.get(), observer->load_page_info()->web_state);
   EXPECT_FALSE(observer->load_page_info()->success);
   web_state_->OnPageLoaded(url, true);
   ASSERT_TRUE(observer->load_page_info());
   EXPECT_EQ(web_state_.get(), observer->load_page_info()->web_state);
   EXPECT_TRUE(observer->load_page_info()->success);
 
-  // Reset the observer and test that DidFinishNavigation() is called
-  // for same document navigations.
+  // Test that OnTitleChanged() is called.
   observer = base::MakeUnique<TestWebStateObserver>(web_state_.get());
-  ASSERT_FALSE(observer->did_finish_navigation_info());
-  web_state_->OnSameDocumentNavigation(url);
-  ASSERT_TRUE(observer->did_finish_navigation_info());
-  EXPECT_EQ(web_state_.get(),
-            observer->did_finish_navigation_info()->web_state);
-  actual_context = observer->did_finish_navigation_info()->context.get();
-  ASSERT_TRUE(actual_context);
-  EXPECT_EQ(url, actual_context->GetUrl());
-  EXPECT_TRUE(actual_context->IsSameDocument());
-  EXPECT_FALSE(actual_context->IsErrorPage());
-  EXPECT_FALSE(actual_context->GetResponseHeaders());
-
-  // Reset the observer and test that DidFinishNavigation() is called
-  // for error navigations.
-  observer = base::MakeUnique<TestWebStateObserver>(web_state_.get());
-  ASSERT_FALSE(observer->did_finish_navigation_info());
-  web_state_->OnErrorPageNavigation(url);
-  ASSERT_TRUE(observer->did_finish_navigation_info());
-  EXPECT_EQ(web_state_.get(),
-            observer->did_finish_navigation_info()->web_state);
-  actual_context = observer->did_finish_navigation_info()->context.get();
-  ASSERT_TRUE(actual_context);
-  EXPECT_EQ(url, actual_context->GetUrl());
-  EXPECT_FALSE(actual_context->IsSameDocument());
-  EXPECT_TRUE(actual_context->IsErrorPage());
-  EXPECT_FALSE(actual_context->GetResponseHeaders());
-
-  // Test that OnTitleChanged() is called.
   ASSERT_FALSE(observer->title_was_set_info());
   web_state_->OnTitleChanged();
   ASSERT_TRUE(observer->title_was_set_info());
   EXPECT_EQ(web_state_.get(), observer->title_was_set_info()->web_state);
 
   // Test that WebStateDestroyed() is called.
   EXPECT_FALSE(observer->web_state_destroyed_info());
   web_state_.reset();
   EXPECT_TRUE(observer->web_state_destroyed_info());
 
@@ skipping 259 matching lines @@
   // Set |created_with_opener| to true and verify that HasOpener() returns true.
   WebState::CreateParams params_with_opener =
       WebState::CreateParams(GetBrowserState());
   params_with_opener.created_with_opener = true;
   std::unique_ptr<WebState> web_state_with_opener =
       WebState::Create(params_with_opener);
   EXPECT_TRUE(web_state_with_opener->HasOpener());
 }
 
 }  // namespace web