GTTF: No more FLAKY_ .

chrome_frame/test/ui_test.cc

 // Copyright (c) 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.
 
 #include <mshtmcid.h>
 #include <string>
 
 #include "base/test/test_file_util.h"
 #include "base/utf_string_conversions.h"
 #include "base/win/scoped_bstr.h"
@@ skipping 96 matching lines @@
                             simulate_input::SHIFT)));
 
   EXPECT_CALL(ie_mock_, OnLoad(in_cf, StrEq(page2)))
       .WillOnce(CloseBrowserMock(&ie_mock_));
 
   LaunchIENavigateAndLoop(page1, kChromeFrameVeryLongNavigationTimeout);
 }
 
 // Tests new window behavior with ctrl+N.
 // Flaky due to DelaySendChar; see http://crbug.com/124244.
-TEST_P(FullTabUITest, FLAKY_CtrlN) {
+TEST_P(FullTabUITest, DISABLED_CtrlN) {
   if (IsWorkstationLocked()) {
     LOG(ERROR) << "This test cannot be run in a locked workstation.";
     return;
   }
 
   bool is_cf = GetParam().invokes_cf();
   if (!is_cf) {
     LOG(ERROR) << "Test not implemented for this configuration.";
     return;
   }
@@ skipping 21 matching lines @@
 
   EXPECT_CALL(win_observer_mock, OnWindowClose(_))
       .Times(testing::AtMost(2));
 
   LaunchIENavigateAndLoop(GetSimplePageUrl(),
                           kChromeFrameVeryLongNavigationTimeout);
 }
 
 // Test that Ctrl+F opens the Find dialog.
 // Flaky due to DelaySendChar; see http://crbug.com/124244.
-TEST_P(FullTabUITest, FLAKY_CtrlF) {
+TEST_P(FullTabUITest, DISABLED_CtrlF) {
   if (IsWorkstationLocked()) {
     LOG(ERROR) << "This test cannot be run in a locked workstation.";
     return;
   }
 
   bool is_cf = GetParam().invokes_cf();
   if (!is_cf) {
     LOG(ERROR) << "Test not implemented for this configuration.";
     return;
   }
@@ skipping 13 matching lines @@
 
   EXPECT_CALL(win_observer_mock, OnWindowOpen(_))
       .WillOnce(CloseBrowserMock(&ie_mock_));
 
   LaunchIENavigateAndLoop(GetSimplePageUrl(),
                           kChromeFrameVeryLongNavigationTimeout);
 }
 
 // Test that ctrl+r does cause a refresh.
 // Flaky due to DelaySendChar; see http://crbug.com/124244.
-TEST_P(FullTabUITest, FLAKY_CtrlR) {
+TEST_P(FullTabUITest, DISABLED_CtrlR) {
   if (IsWorkstationLocked()) {
     LOG(ERROR) << "This test cannot be run in a locked workstation.";
     return;
   }
 
   EXPECT_CALL(server_mock_, Get(_, UrlPathEq(GetSimplePageUrl()), _))
       .Times(testing::AtMost(2))
       .WillRepeatedly(SendResponse(&server_mock_, GetParam()));
 
   EXPECT_CALL(ie_mock_, OnLoad(GetParam().invokes_cf(),
                                StrEq(GetSimplePageUrl())))
       .Times(testing::AtMost(2))
       .WillOnce(testing::DoAll(
           SetFocusToRenderer(&ie_mock_),
           DelaySendChar(&loop_,
                         base::TimeDelta::FromSeconds(1),
                         'r',
                         simulate_input::CONTROL),
           DelayCloseBrowserMock(
               &loop_, base::TimeDelta::FromSeconds(4), &ie_mock_)))
       .WillRepeatedly(testing::Return());
 
   LaunchIENavigateAndLoop(GetSimplePageUrl(),
                           kChromeFrameVeryLongNavigationTimeout);
 }
 
 // Test window close with ctrl+w.
 // Flaky due to DelaySendChar; see http://crbug.com/124244.
-TEST_P(FullTabUITest, FLAKY_CtrlW) {
+TEST_P(FullTabUITest, DISABLED_CtrlW) {
   if (IsWorkstationLocked()) {
     LOG(ERROR) << "This test cannot be run in a locked workstation.";
     return;
   }
 
   EXPECT_CALL(ie_mock_, OnLoad(GetParam().invokes_cf(),
                                StrEq(GetSimplePageUrl())))
       .WillOnce(testing::DoAll(
           SetFocusToRenderer(&ie_mock_),
           DelaySendChar(&loop_,
                         base::TimeDelta::FromSeconds(1),
                         'w',
                         simulate_input::CONTROL)));
 
   LaunchIENavigateAndLoop(GetSimplePageUrl(),
                           kChromeFrameVeryLongNavigationTimeout);
 }
 
 // Test address bar navigation with Alt+d and URL.
 // Flaky due to TypeUrlInAddressBar; see http://crbug.com/124244.
-TEST_P(FullTabUITest, FLAKY_AltD) {
+TEST_P(FullTabUITest, DISABLED_AltD) {
   if (IsWorkstationLocked()) {
     LOG(ERROR) << "This test cannot be run in a locked workstation.";
     return;
   }
 
   EXPECT_CALL(ie_mock_, OnLoad(GetParam().invokes_cf(),
                                StrEq(GetSimplePageUrl())))
       .WillOnce(testing::DoAll(
           SetFocusToRenderer(&ie_mock_),
           TypeUrlInAddressBar(&loop_,
@@ skipping 548 matching lines @@
           AccLeftClick(AccObjectMatcher(L"Copy link address*")),
           CloseBrowserMock(&ie_mock_)));
 
   LaunchIEAndNavigate(GetLinkPageUrl());
 
   EXPECT_STREQ(GetSimplePageUrl().c_str(), GetClipboardText().c_str());
 }
 
 // Test CF text field context menu - cut.
 // Times out sporadically http://crbug.com/119660.
-TEST_F(ContextMenuTest, FLAKY_CFTxtFieldCut) {
+TEST_F(ContextMenuTest, DISABLED_CFTxtFieldCut) {
   server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag());
   AccObjectMatcher txtfield_matcher(L"", L"editable text");
 
   // Invoke "Cut" context menu item of text field.
   EXPECT_CALL(acc_observer_,
               OnAccDocLoad(TabContentsTitleEq(context_menu_page_title)))
       .WillOnce(testing::DoAll(
           AccRightClick(txtfield_matcher),
           AccWatchForOneValueChange(&acc_observer_, txtfield_matcher)));
   EXPECT_CALL(acc_observer_, OnMenuPopup(_))
     .WillOnce(AccLeftClick(AccObjectMatcher(L"Cut*")));
 
   // Verify that text field is empty after cut operation.
   EXPECT_CALL(acc_observer_, OnAccValueChange(_, _, StrEq(L"")))
       .WillOnce(CloseBrowserMock(&ie_mock_));
 
   LaunchIEAndNavigate(context_menu_page_url);
   // Verify that the text value has been cut to clipboard.
   EXPECT_STREQ(kTextFieldInitValue.c_str(), GetClipboardText().c_str());
 }
 
 // Test CF text field context menu - copy.
 // Times out sporadically http://crbug.com/119660.
-TEST_F(ContextMenuTest, FLAKY_CFTxtFieldCopy) {
+TEST_F(ContextMenuTest, DISABLED_CFTxtFieldCopy) {
   server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag());
   AccObjectMatcher txtfield_matcher(L"", L"editable text");
 
   // Invoke "Copy" context menu item of text field.
   EXPECT_CALL(acc_observer_,
               OnAccDocLoad(TabContentsTitleEq(context_menu_page_title)))
       .WillOnce(testing::DoAll(
           AccRightClick(txtfield_matcher),
           AccWatchForOneValueChange(&acc_observer_, txtfield_matcher)));
   EXPECT_CALL(acc_observer_, OnMenuPopup(_))
     .WillOnce(testing::DoAll(
         AccLeftClick(AccObjectMatcher(L"Copy*")),
         CloseBrowserMock(&ie_mock_)));
 
   // Verify that there is no change on text field value after copy operation.
   EXPECT_CALL(acc_observer_, OnAccValueChange(_, _, _))
       .Times(testing::AtMost(0));
 
   LaunchIEAndNavigate(context_menu_page_url);
   // Verify that the text value has been copied to clipboard.
   EXPECT_STREQ(kTextFieldInitValue.c_str(), GetClipboardText().c_str());
 }
 
 // Test CF text field context menu - paste.
 // Times out sporadically http://crbug.com/119660.
-TEST_F(ContextMenuTest, FLAKY_CFTxtFieldPaste) {
+TEST_F(ContextMenuTest, DISABLED_CFTxtFieldPaste) {
   server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag());
   AccObjectMatcher txtfield_matcher(L"", L"editable text");
 
   // Invoke "Paste" context menu item of text field.
   EXPECT_CALL(acc_observer_,
               OnAccDocLoad(TabContentsTitleEq(context_menu_page_title)))
       .WillOnce(testing::DoAll(
           AccRightClick(txtfield_matcher),
           AccWatchForOneValueChange(&acc_observer_, txtfield_matcher)));
   EXPECT_CALL(acc_observer_, OnMenuPopup(_))
       .WillOnce(AccLeftClick(AccObjectMatcher(L"Paste*")));
   // Verify that value has been pasted to text field.
   EXPECT_CALL(acc_observer_, OnAccValueChange(_, _, StrEq(kTextFieldInitValue)))
       .WillOnce(CloseBrowserMock(&ie_mock_));
 
   // Set some text value to clipboard, this is to emulate the 'copy' action.
   SetClipboardText(kTextFieldInitValue);
 
   LaunchIEAndNavigate(context_menu_page_url);
 }
 
 // Test CF text field context menu - delete.
 // Times out sporadically http://crbug.com/119660.
-TEST_F(ContextMenuTest, FLAKY_CFTxtFieldDelete) {
+TEST_F(ContextMenuTest, DISABLED_CFTxtFieldDelete) {
   server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag());
   AccObjectMatcher txtfield_matcher(L"", L"editable text");
 
   // Invoke 'Delete' context menu item of text field.
   EXPECT_CALL(acc_observer_,
               OnAccDocLoad(TabContentsTitleEq(context_menu_page_title)))
       .WillOnce(testing::DoAll(
           AccRightClick(txtfield_matcher),
           AccWatchForOneValueChange(&acc_observer_, txtfield_matcher)));
   EXPECT_CALL(acc_observer_, OnMenuPopup(_))
@@ skipping 21 matching lines @@
   // Client side script verifies that the text field value has been selected,
   // then send 'OK' message.
   EXPECT_CALL(ie_mock_, OnMessage(testing::StrCaseEq(L"OK"), _, _))
       .WillOnce(CloseBrowserMock(&ie_mock_));
 
   LaunchIEAndNavigate(context_menu_page_url + L"?action=selectall");
 }
 
 // Test CF text field context menu - undo.
 // Times out sporadically http://crbug.com/119660.
-TEST_F(ContextMenuTest, FLAKY_CFTxtFieldUndo) {
+TEST_F(ContextMenuTest, DISABLED_CFTxtFieldUndo) {
   server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag());
   AccObjectMatcher txtfield_matcher(L"", L"editable text");
 
   // Change the value of text field to 'A'.
   EXPECT_CALL(acc_observer_,
               OnAccDocLoad(TabContentsTitleEq(context_menu_page_title)))
       .WillOnce(testing::DoAll(
           AccWatchForOneValueChange(&acc_observer_, txtfield_matcher),
           AccSendCharMessage(txtfield_matcher, L'A')));
   // Bring up the context menu once the value has changed.
   EXPECT_CALL(acc_observer_, OnAccValueChange(_, _, StrEq(L"A")))
       .WillOnce(AccRightClick(txtfield_matcher));
   // Then select "Undo".
   EXPECT_CALL(acc_observer_, OnMenuPopup(_))
       .WillOnce(testing::DoAll(
           AccWatchForOneValueChange(&acc_observer_, txtfield_matcher),
           AccLeftClick(AccObjectMatcher(L"Undo*"))));
 
   // Verify that value has been reset to initial value after undo operation.
   EXPECT_CALL(acc_observer_, OnAccValueChange(_, _, StrEq(kTextFieldInitValue)))
       .WillOnce(CloseBrowserMock(&ie_mock_));
 
   LaunchIEAndNavigate(context_menu_page_url);
 }
 
 // Test CF text field context menu - redo.
 // Times out sporadically http://crbug.com/119660.
-TEST_F(ContextMenuTest, FLAKY_CFTxtFieldRedo) {
+TEST_F(ContextMenuTest, DISABLED_CFTxtFieldRedo) {
   server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag());
   AccObjectMatcher txtfield_matcher(L"", L"editable text");
   InSequence expect_in_sequence_for_scope;
 
   // Change text field from its initial value to 'A'.
   EXPECT_CALL(acc_observer_,
               OnAccDocLoad(TabContentsTitleEq(context_menu_page_title)))
       .WillOnce(testing::DoAll(
           AccWatchForOneValueChange(&acc_observer_, txtfield_matcher),
           AccSendCharMessage(txtfield_matcher, L'A')));
@@ skipping 91 matching lines @@
   EXPECT_CALL(acc_observer_, OnMenuPopup(_))
       .WillOnce(AccLeftClick(AccObjectMatcher(L"Forward")));
 
   EXPECT_CALL(ie_mock_, OnLoad(IN_CF, StrEq(page3)))
       .WillOnce(CloseBrowserMock(&ie_mock_));
 
   LaunchIENavigateAndLoop(page1, kChromeFrameVeryLongNavigationTimeout);
 }
 
 }  // namespace chrome_frame_test