Fix SitePerProcessTextInputManagerTest.StopTrackingCrashedChildFrame on CFI Bots.

chrome/browser/renderer_host/site_per_process_text_input_browsertest.cc

 // Copyright 2016 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.
 
 #include <vector>
 
 #include "base/command_line.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/test/base/in_process_browser_test.h"
@@ skipping 305 matching lines @@
   // Make sure that the first view has set its TextInputState.type to NONE.
   view_type_observer.Wait();
 
   // Verify that we are tracking the TextInputState from the first frame.
   content::RenderWidgetHostView* first_view = frames[0]->GetView();
   ui::TextInputType first_view_type;
   EXPECT_TRUE(content::GetTextInputTypeForView(active_contents(), first_view,
                                                &first_view_type));
   EXPECT_EQ(ui::TEXT_INPUT_TYPE_NONE, first_view_type);
 
+  size_t registered_views_count =

[EhsanK, 2016/06/11 10:26:56]

I was expecting this number to be 2U as it was in most bots. But on Linux for
some reason we receive a TextInputStateChanged from perhaps the tab's RWHV (so
the number is 3). It is unrelated to this test and browser side tracking
altogether. 

+      content::GetAllViewsRegisteredWithTextInputManager(active_contents())

[Charlie Reis, 2016/06/13 18:46:13]

We should just expose the count if we don't need to look at the views themselves
(GetRegisteredViewCountFromTextInputManager).

[EhsanK, 2016/06/25 00:10:34]

Acknowledged.

+          .size();
+
   // Now that the second frame's <input> is focused, we crash the first frame
   // and observe that text input state is updated for the view.
   std::unique_ptr<content::TestRenderWidgetHostViewDestructionObserver>
       destruction_observer(
           new content::TestRenderWidgetHostViewDestructionObserver(first_view));
   frames[0]->GetProcess()->Shutdown(0, false);
   destruction_observer->Wait();
 
   // Verifying that the TextInputManager is no longer tracking TextInputState
-  // for |first_view|. Note that |first_view| is now a dangling pointer.
-  EXPECT_FALSE(content::GetTextInputTypeForView(active_contents(), first_view,
-                                                &first_view_type));
+  // for |first_view|.
+  EXPECT_EQ(
+      registered_views_count - 1U,
+      content::GetAllViewsRegisteredWithTextInputManager(active_contents())
+          .size());
 
   // Now crash the second <iframe> which has an active view.
-  content::RenderWidgetHostView* second_view = frames[1]->GetView();
   TextInputManagerChangeObserver change_observer(active_contents());
   frames[1]->GetProcess()->Shutdown(0, false);
   change_observer.Wait();
   EXPECT_EQ(ui::TEXT_INPUT_TYPE_NONE,
             content::GetTextInputTypeFromWebContents(active_contents()));
   EXPECT_FALSE(!!content::GetActiveViewFromWebContents(active_contents()));
-  ui::TextInputType second_view_type;
-  EXPECT_FALSE(content::GetTextInputTypeForView(active_contents(), second_view,
-                                                &second_view_type));
+  EXPECT_EQ(
+      registered_views_count - 2U,
+      content::GetAllViewsRegisteredWithTextInputManager(active_contents())
+          .size());
 }
 
 // The following test loads a page with two child frames: one in process and one
 // out of process with main frame. The test inserts an <input> inside each frame
 // and focuses the first frame and observes the TextInputManager setting the
 // state to ui::TEXT_INPUT_TYPE_TEXT. Then, the frame is detached and the test
 // observes that the state type is reset to ui::TEXT_INPUT_TYPE_NONE. The same
 // sequence of actions is then performed on the out of process frame.
 IN_PROC_BROWSER_TEST_F(SitePerProcessTextInputManagerTest,
                        ResetStateAfterFrameDetached) {
@@ skipping 188 matching lines @@
   EXPECT_FALSE(send_and_check_show_ime());
 
   // Set |TextInputState.show_ime_if_needed|. Expect IME.
   sender.SetShowImeIfNeeded(true);
   EXPECT_TRUE(send_and_check_show_ime());
 
   // Set |TextInputState.type| to ui::TEXT_INPUT_TYPE_NONE. Expect no IME.
   sender.SetType(ui::TEXT_INPUT_TYPE_NONE);
   EXPECT_FALSE(send_and_check_show_ime());
 }