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

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

Issue 7069026: Disabling the ContextMenuTest.CFSavePageAs test as the attempted fix did not work. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 7 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 | « no previous file | no next file » | 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) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 547 matching lines...) Expand 10 before | Expand all | Expand 10 after
558 // for more information on why this test is disabled for Vista with IE7. 558 // for more information on why this test is disabled for Vista with IE7.
559 if (base::win::GetVersion() == base::win::VERSION_VISTA && 559 if (base::win::GetVersion() == base::win::VERSION_VISTA &&
560 GetInstalledIEVersion() == IE_7) { 560 GetInstalledIEVersion() == IE_7) {
561 LOG(INFO) << "Not running test on Vista with IE7"; 561 LOG(INFO) << "Not running test on Vista with IE7";
562 return; 562 return;
563 } 563 }
564 ASSERT_NO_FATAL_FAILURE(DoSaveAsTest(L"", L"Save as...", L".html")); 564 ASSERT_NO_FATAL_FAILURE(DoSaveAsTest(L"", L"Save as...", L".html"));
565 } 565 }
566 566
567 // http://code.google.com/p/chromium/issues/detail?id=83114 567 // http://code.google.com/p/chromium/issues/detail?id=83114
568 TEST_F(ContextMenuTest, FLAKY_CFSaveLinkAs) { 568 TEST_F(ContextMenuTest, DISABLED_CFSaveLinkAs) {
569 // Please see http://code.google.com/p/chromium/issues/detail?id=60987 569 // Please see http://code.google.com/p/chromium/issues/detail?id=60987
570 // for more information on why this test is disabled for Vista with IE7. 570 // for more information on why this test is disabled for Vista with IE7.
571 if (base::win::GetVersion() == base::win::VERSION_VISTA && 571 if (base::win::GetVersion() == base::win::VERSION_VISTA &&
572 GetInstalledIEVersion() == IE_7) { 572 GetInstalledIEVersion() == IE_7) {
573 LOG(INFO) << "Not running test on Vista with IE7"; 573 LOG(INFO) << "Not running test on Vista with IE7";
574 return; 574 return;
575 } 575 }
576 ASSERT_NO_FATAL_FAILURE(DoSaveAsTest(L"link", L"Save link as...", L".zip")); 576 ASSERT_NO_FATAL_FAILURE(DoSaveAsTest(L"link", L"Save link as...", L".zip"));
577 } 577 }
578 578
(...skipping 387 matching lines...) Expand 10 before | Expand all | Expand 10 after
966 .WillOnce(AccLeftClick(AccObjectMatcher(L"Forward"))); 966 .WillOnce(AccLeftClick(AccObjectMatcher(L"Forward")));
967 967
968 EXPECT_CALL(ie_mock_, OnLoad(IN_CF, StrEq(page3))) 968 EXPECT_CALL(ie_mock_, OnLoad(IN_CF, StrEq(page3)))
969 .WillOnce(CloseBrowserMock(&ie_mock_)); 969 .WillOnce(CloseBrowserMock(&ie_mock_));
970 970
971 LaunchIENavigateAndLoop(page1, 971 LaunchIENavigateAndLoop(page1,
972 kChromeFrameVeryLongNavigationTimeoutInSeconds); 972 kChromeFrameVeryLongNavigationTimeoutInSeconds);
973 } 973 }
974 974
975 } // namespace chrome_frame_test 975 } // namespace chrome_frame_test
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698