Revert of Reset Text Input State for RenderWidgetHostView before RenderWidgetHost Detaches from ...

chrome/browser/apps/guest_view/web_view_interactive_browsertest.cc

 // Copyright 2013 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 <stddef.h>
 
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/stringprintf.h"
@@ skipping 1397 matching lines @@
       embedder_web_contents(), get_type_checker(ui::TEXT_INPUT_TYPE_TEXT));
 
   // Press the tab key one more time to get back to embedder's second <input>.
   // The value should be "last one".
   press_tab_to_focus(this, "EMBEDDER-FOCUSED-2");
   TextInputStateHelper::WaitForDesiredState(
       embedder_web_contents(),
       base::Bind(&TextInputStateHelper::HasGivenValue, "last one"));
 }
 
+// TODO(ekaramad): Activate this test for OOPIF when input event routing for
+// OOPIF-<webview> is fixed.
 IN_PROC_BROWSER_TEST_P(WebViewTextInputStateInteractiveTest,
                        CrashingWebViewResetsState) {
   SetupTest("web_view/text_input_state",
             "/extensions/platform_apps/web_view/text_input_state/guest.html");
 
-  // Press tab key twice to end up in the <input> of the <webview>.
+  // Press tab key twice to end up in the <input> of the <webview>,
   ExtensionTestMessageListener listener("GUEST-FOCUSED", false);
   for (size_t i = 0; i < 2; ++i)
     SendKeyPressToPlatformApp(ui::VKEY_TAB);
 
   listener.WaitUntilSatisfied();
 
   // Now wait for a text input state change.
   TextInputStateHelper::WaitForDesiredState(
       embedder_web_contents(),
       base::Bind(&TextInputStateHelper::HasGivenValue, "guest"));
 
   // Now crash the <webview>.
   guest_web_contents()->GetRenderProcessHost()->Shutdown(false, 0);
 
-  // Wait for the outer WebContentsImpl |text_input_state_->type| to be reset to
-  // none.
+  // State should reset to none.
   TextInputStateHelper::WaitForDesiredState(
       embedder_web_contents(),
       base::Bind(&TextInputStateHelper::IsStateOfGivenType,
                  ui::TEXT_INPUT_TYPE_NONE));
 }
-
-// This test creates a <webview> with a text input field inside, gives focus to
-// the input field, and then detaches the <webview>. It monitors the embedder
-// WebContents text input state to make sure it tracks the state properly.
-IN_PROC_BROWSER_TEST_P(WebViewTextInputStateInteractiveTest,
-                       OuterWebContentsResetsStateAfterDetach) {
-  SetupTest("web_view/text_input_state",
-            "/extensions/platform_apps/web_view/text_input_state/guest.html");
-
-  // Press tab key twice to end up in the <input> of the <webview>.
-  ExtensionTestMessageListener listener("GUEST-FOCUSED", false);
-  for (size_t i = 0; i < 2; ++i)
-    SendKeyPressToPlatformApp(ui::VKEY_TAB);
-
-  listener.WaitUntilSatisfied();
-
-  // Now wait for a text input state change.
-  TextInputStateHelper::WaitForDesiredState(
-      embedder_web_contents(),
-      base::Bind(&TextInputStateHelper::HasGivenValue, "guest"));
-
-  // Now detach the <webview>.
-  ExtensionTestMessageListener detach_listener("detached", false);
-  detach_listener.set_failure_message("failed-to-detach");
-  EXPECT_TRUE(
-      content::ExecuteScript(embedder_web_contents(), "detachWebView();"));
-  detach_listener.WaitUntilSatisfied();
-
-  // Wait for the outer WebContentsImpl |text_input_state_->type| to be reset to
-  // none.
-  TextInputStateHelper::WaitForDesiredState(
-      embedder_web_contents(),
-      base::Bind(&TextInputStateHelper::IsStateOfGivenType,
-                 ui::TEXT_INPUT_TYPE_NONE));
-}