Removed -[CRWWebDelegate webController:titleDidChange:].

ios/web/web_state/ui/crw_web_controller_unittest.mm

 // Copyright 2012 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/ui/crw_web_controller.h"
 
 #import <WebKit/WebKit.h>
 
 #include <utility>
 
 #include "base/ios/ios_util.h"
 #import "base/mac/scoped_nsobject.h"
+#include "base/strings/utf_string_conversions.h"
 #import "base/test/ios/wait_util.h"
 #import "ios/testing/ocmock_complex_type_helper.h"
 #import "ios/web/navigation/crw_session_controller.h"
 #import "ios/web/navigation/crw_session_entry.h"
 #import "ios/web/navigation/navigation_item_impl.h"
 #import "ios/web/navigation/navigation_manager_impl.h"
 #include "ios/web/public/referrer.h"
 #import "ios/web/public/test/fakes/test_native_content.h"
 #import "ios/web/public/test/fakes/test_native_content_provider.h"
 #import "ios/web/public/test/fakes/test_web_client.h"
@@ skipping 962 matching lines @@
 TEST_F(CRWWebControllerWindowOpenTest, BlockPopup) {
   ASSERT_FALSE([web_controller() userIsInteracting]);
   EXPECT_NSEQ([NSNull null], OpenWindowByDOM());
 
   EXPECT_EQ(web_controller(), [delegate_ webController]);
   EXPECT_EQ("javascript:void(0);", [delegate_ childURL].spec());
   EXPECT_EQ("http://test/", [delegate_ openerURL].spec());
   EXPECT_FALSE([delegate_ initiatedByUser]);
 };
 
+// Tests page title changes.
+typedef web::WebTestWithWebState CRWWebControllerTitleTest;
+TEST_F(CRWWebControllerTitleTest, TitleChange) {
+  // Observes and waits for TitleWasSet call.
+  class TitleObserver : public web::WebStateObserver {
+   public:
+    explicit TitleObserver(web::WebState* web_state)
+        : web::WebStateObserver(web_state) {}
+    // Returns number of times |TitleWasSet| was called.
+    int title_change_count() { return title_change_count_; }
+    // WebStateObserver overrides:
+    void TitleWasSet() override { title_change_count_++; }
+
+   private:
+    int title_change_count_ = 0;
+  };
+
+  TitleObserver observer(web_state());
+  ASSERT_EQ(0, observer.title_change_count());
+
+  // Expect TitleWasSet callback after the page is loaded.
+  LoadHtml(@"<title>Title1</title>");
+  EXPECT_EQ("Title1", base::UTF16ToUTF8(web_state()->GetTitle()));
+  EXPECT_EQ(1, observer.title_change_count());
+
+  // Expect at least one more TitleWasSet callback after changing title via
+  // JavaScript. On iOS 10 WKWebView fires 3 callbacks after JS excucution
+  // with the following title changes: "Title2", "" and "Title2".
+  // TODO(crbug.com/696104): There should be only 2 calls of TitleWasSet.
+  // Fix expecteation when WKWebView stops sending extra KVO calls.
+  ExecuteJavaScript(@"window.document.title = 'Title2';");
+  EXPECT_EQ("Title2", base::UTF16ToUTF8(web_state()->GetTitle()));
+  EXPECT_GE(observer.title_change_count(), 2);
+};
+
 // Fixture class to test WKWebView crashes.
 class CRWWebControllerWebProcessTest : public web::WebTestWithWebController {
  protected:
   void SetUp() override {
     web::WebTestWithWebController::SetUp();
     webView_.reset([web::BuildTerminatedWKWebView() retain]);
     base::scoped_nsobject<TestWebViewContentView> webViewContentView(
         [[TestWebViewContentView alloc]
             initWithMockWebView:webView_
                      scrollView:[webView_ scrollView]]);
@@ skipping 26 matching lines @@
   };
 
   RenderProcessGoneObserver observer(web_state());
   web::SimulateWKWebViewCrash(webView_);
   observer.WaitForRenderProcessGone();
 
   EXPECT_FALSE([web_controller() isViewAlive]);
 };
 
 }  // namespace