Disable two flaky tests in SitePerProcessTextInputManagerTest

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 "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_content_browser_client.h"
 #include "chrome/browser/ui/browser.h"
@@ skipping 853 matching lines @@
                      ui::DomCode::US_E, ui::VKEY_E, false, false, false, false);
     bounds_observer.Wait();
   };
 
   for (auto* view : views)
     send_tab_insert_text_wait_for_bounds_change(view);
 }
 
 // This test makes sure browser correctly tracks focused editable element inside
 // each RenderFrameHost.
-// Test is flaky on ChromeOS. crbug.com/705203
-#if defined(OS_CHROMEOS)
-#define MAYBE_TrackingFocusedElementForAllFrames \
-  DISABLED_TrackingFocusedElementForAllFrames
-#else
-#define MAYBE_TrackingFocusedElementForAllFrames \
-  TrackingFocusedElementForAllFrames
-#endif
+// Test is flaky. crbug.com/705203
 IN_PROC_BROWSER_TEST_F(SitePerProcessTextInputManagerTest,
-                       MAYBE_TrackingFocusedElementForAllFrames) {
+                       DISABLED_TrackingFocusedElementForAllFrames) {
   CreateIframePage("a(a, b(a))");
   std::vector<content::RenderFrameHost*> frames{
       GetFrame(IndexVector{}), GetFrame(IndexVector{0}),
       GetFrame(IndexVector{1}), GetFrame(IndexVector{1, 0})};
   for (size_t i = 0; i < frames.size(); ++i)
     AddInputFieldToFrame(frames[i], "text", "some text", true);
 
   // Focus the <input> in |frame| and return if RenderFrameHost thinks there is
   // a focused editable element in it.
   auto focus_input_and_return_editable_element_state =
@@ skipping 61 matching lines @@
     set_input_focus(frame, false);
     EXPECT_FALSE(active_contents()->IsFocusedElementEditable());
   }
 }
 
 // TODO(ekaramad): Could this become a unit test instead?
 // This test focuses <input> elements on the page and verifies that
 // WebContents knows about the focused editable element. Then it asks the
 // WebContents to clear focused element and verifies that there is no longer
 // a focused editable element on the page.
+// Test is flaky. crbug.com/705203
 IN_PROC_BROWSER_TEST_F(SitePerProcessTextInputManagerTest,
-                       ClearFocusedElementOnPage) {
+                       DISABLED_ClearFocusedElementOnPage) {
   CreateIframePage("a(a, b(a))");
   std::vector<content::RenderFrameHost*> frames{
       GetFrame(IndexVector{}), GetFrame(IndexVector{0}),
       GetFrame(IndexVector{1}), GetFrame(IndexVector{1, 0})};
   for (size_t i = 0; i < frames.size(); ++i)
     AddInputFieldToFrame(frames[i], "text", "some text", true);
 
   auto focus_frame_and_input = [](content::RenderFrameHost* frame) {
     EXPECT_TRUE(ExecuteScript(frame,
                               "window.focus();"
@@ skipping 394 matching lines @@
 
   // Closing this WebContents while we still hold on to our TestBrowserClient.
   EXPECT_TRUE(browser()->tab_strip_model()->CloseWebContentsAt(
       1, TabStripModel::CLOSE_USER_GESTURE));
 
   // For the cleanup of the original WebContents in tab index 0.
   content::SetBrowserClientForTesting(old_browser_client);
 }
 #endif  //  defined(MAC_OSX)
 #endif  // !defined(OS_ANDROID)