Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(381)

Side by Side Diff: chrome_frame/test/ui_test.cc

Issue 9460019: Reduce flakiness in chrome_frame_tests.exe by having each run in a clean environment. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixed compile break Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome_frame/test/test_with_web_server.cc ('k') | chrome_frame/test_utils.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <mshtmcid.h> 5 #include <mshtmcid.h>
6 #include <string> 6 #include <string>
7 7
8 #include "base/test/test_file_util.h" 8 #include "base/test/test_file_util.h"
9 #include "base/utf_string_conversions.h" 9 #include "base/utf_string_conversions.h"
10 #include "base/win/scoped_bstr.h" 10 #include "base/win/scoped_bstr.h"
(...skipping 785 matching lines...) Expand 10 before | Expand all | Expand 10 after
796 // Verify that text field is empty after cut operation. 796 // Verify that text field is empty after cut operation.
797 EXPECT_CALL(acc_observer_, OnAccValueChange(_, _, StrEq(L""))) 797 EXPECT_CALL(acc_observer_, OnAccValueChange(_, _, StrEq(L"")))
798 .WillOnce(CloseBrowserMock(&ie_mock_)); 798 .WillOnce(CloseBrowserMock(&ie_mock_));
799 799
800 LaunchIEAndNavigate(context_menu_page_url); 800 LaunchIEAndNavigate(context_menu_page_url);
801 // Verify that the text value has been cut to clipboard. 801 // Verify that the text value has been cut to clipboard.
802 EXPECT_STREQ(kTextFieldInitValue.c_str(), GetClipboardText().c_str()); 802 EXPECT_STREQ(kTextFieldInitValue.c_str(), GetClipboardText().c_str());
803 } 803 }
804 804
805 // Test CF text field context menu - copy. 805 // Test CF text field context menu - copy.
806 // FLAKY: 114386. 806 TEST_F(ContextMenuTest, CFTxtFieldCopy) {
807 TEST_F(ContextMenuTest, DISABLED_CFTxtFieldCopy) {
808 server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag()); 807 server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag());
809 AccObjectMatcher txtfield_matcher(L"", L"editable text"); 808 AccObjectMatcher txtfield_matcher(L"", L"editable text");
810 809
811 // Invoke "Copy" context menu item of text field. 810 // Invoke "Copy" context menu item of text field.
812 EXPECT_CALL(acc_observer_, 811 EXPECT_CALL(acc_observer_,
813 OnAccDocLoad(TabContentsTitleEq(context_menu_page_title))) 812 OnAccDocLoad(TabContentsTitleEq(context_menu_page_title)))
814 .WillOnce(testing::DoAll( 813 .WillOnce(testing::DoAll(
815 AccRightClick(txtfield_matcher), 814 AccRightClick(txtfield_matcher),
816 AccWatchForOneValueChange(&acc_observer_, txtfield_matcher))); 815 AccWatchForOneValueChange(&acc_observer_, txtfield_matcher)));
817 EXPECT_CALL(acc_observer_, OnMenuPopup(_)) 816 EXPECT_CALL(acc_observer_, OnMenuPopup(_))
(...skipping 205 matching lines...) Expand 10 before | Expand all | Expand 10 after
1023 .WillOnce(AccLeftClick(AccObjectMatcher(L"Forward"))); 1022 .WillOnce(AccLeftClick(AccObjectMatcher(L"Forward")));
1024 1023
1025 EXPECT_CALL(ie_mock_, OnLoad(IN_CF, StrEq(page3))) 1024 EXPECT_CALL(ie_mock_, OnLoad(IN_CF, StrEq(page3)))
1026 .WillOnce(CloseBrowserMock(&ie_mock_)); 1025 .WillOnce(CloseBrowserMock(&ie_mock_));
1027 1026
1028 LaunchIENavigateAndLoop(page1, 1027 LaunchIENavigateAndLoop(page1,
1029 kChromeFrameVeryLongNavigationTimeoutInSeconds); 1028 kChromeFrameVeryLongNavigationTimeoutInSeconds);
1030 } 1029 }
1031 1030
1032 } // namespace chrome_frame_test 1031 } // namespace chrome_frame_test
OLDNEW
« no previous file with comments | « chrome_frame/test/test_with_web_server.cc ('k') | chrome_frame/test_utils.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698