[refactor] Removing the dead method RenderWidgetHostView::GetSelectedText()

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 1267 matching lines @@
     EXPECT_TRUE(content::ExecuteScript(
                     embedder_web_contents,
                     "window.runCommand('testInputMethodRunNextStep', 3);"));
 
     // Wait for the next step to complete.
     ASSERT_TRUE(next_step_listener.WaitUntilSatisfied());
   }
 }
 #endif
 
-#if defined(OS_MACOSX)
-IN_PROC_BROWSER_TEST_P(WebViewInteractiveTest, TextSelection) {

[EhsanK, 2016/07/13 17:44:57]

This is just testing that RWHVGuest::GetSelectedText works which as we saw is
not used at all. I guess it is OK to remove this test.

-#if defined(OS_MACOSX)
-  // TODO(ekaramad): This test is failing under OOPIF for MAC.
-  if (GetParam())
-    return;
-#endif
-  SetupTest("web_view/text_selection",
-            "/extensions/platform_apps/web_view/text_selection/guest.html");
-  ASSERT_TRUE(guest_web_contents());
-  ASSERT_TRUE(ui_test_utils::ShowAndFocusNativeWindow(
-      GetPlatformAppWindow()));
-
-  // Wait until guest sees a context menu, select an arbitrary item (copy).
-  ExtensionTestMessageListener ctx_listener("MSG_CONTEXTMENU", false);
-  ContextMenuNotificationObserver menu_observer(IDC_CONTENT_CONTEXT_COPY);
-  SimulateRWHMouseClick(guest_web_contents()->GetRenderViewHost()->GetWidget(),
-                        blink::WebMouseEvent::ButtonRight, 20, 20);
-  ASSERT_TRUE(ctx_listener.WaitUntilSatisfied());
-
-  // Now verify that the selection text propagates properly to RWHV.
-  content::RenderWidgetHostView* guest_rwhv =
-      guest_web_contents()->GetRenderWidgetHostView();
-  ASSERT_TRUE(guest_rwhv);
-  std::string selected_text = base::UTF16ToUTF8(guest_rwhv->GetSelectedText());
-  ASSERT_TRUE(selected_text.size() >= 10u);
-  ASSERT_EQ("AAAAAAAAAA", selected_text.substr(0, 10));
-}
-#endif
-
 IN_PROC_BROWSER_TEST_F(WebViewFocusInteractiveTest, FocusAndVisibility) {
   ASSERT_TRUE(StartEmbeddedTestServer());
   LoadAndLaunchPlatformApp("web_view/focus_visibility",
                            "WebViewInteractiveTest.LOADED");
   ExtensionTestMessageListener test_init_listener(
       "WebViewInteractiveTest.WebViewInitialized", false);
   SendMessageToEmbedder("init");
   test_init_listener.WaitUntilSatisfied();
   // Send several tab-keys. The button inside webview should receive focus at
   // least once.
@@ skipping 18 matching lines @@
   // this time.
   for (size_t i = 0; i < 4; ++i)
     SendKeyPressToPlatformApp(ui::VKEY_TAB);
   ExtensionTestMessageListener webview_button_not_focused_listener(
       "WebViewInteractiveTest.WebViewButtonWasNotFocused", false);
   webview_button_not_focused_listener.set_failure_message(
       "WebViewInteractiveTest.WebViewButtonWasFocused");
   SendMessageToEmbedder("verify");
   EXPECT_TRUE(webview_button_not_focused_listener.WaitUntilSatisfied());
 }