Disabling a number of failing chrome frame tests on Vista with IE7. Most of t...

chrome_frame/test/navigation_test.cc

 // Copyright (c) 2010 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 <string>
 
 #include "base/scoped_comptr_win.h"
 #include "base/test/test_file_util.h"
+#include "base/win/windows_version.h"
 #include "chrome_frame/test/chrome_frame_test_utils.h"
 #include "chrome_frame/test/chrome_frame_ui_test_utils.h"
 #include "chrome_frame/test/mock_ie_event_sink_actions.h"
 #include "chrome_frame/test/mock_ie_event_sink_test.h"
 
 // Needed for CreateFunctor.
 #define GMOCK_MUTANT_INCLUDE_LATE_OBJECT_BINDING
 #include "testing/gmock_mutant.h"
 
 using testing::_;
@@ skipping 354 matching lines @@
 
   // OnNavigateError can take a long time to fire.
   LaunchIENavigateAndLoop(parent_url,
                           kChromeFrameLongNavigationTimeoutInSeconds * 4);
   ASSERT_TRUE(new_window_mock.event_sink()->web_browser2() != NULL);
 }
 
 // Tests that the parent window can successfully close its popup through
 // the javascript close method.
 TEST_P(FullTabNavigationTest, JavascriptWindowOpenCanClose) {
+  // Please see http://code.google.com/p/chromium/issues/detail?id=60987
+  // for more information on why this test is disabled for Vista with IE7.
+  if (base::win::GetVersion() == base::win::VERSION_VISTA &&
+      GetInstalledIEVersion() == IE_7) {
+    LOG(INFO) << "Not running test on Vista with IE7";
+    return;
+  }
+
   std::wstring parent_url = GetTestUrl(L"window_open.html?simple.html");
   MockAccEventObserver acc_observer;
   MockIEEventSink new_window_mock;
   ie_mock_.ExpectAnyNavigations();
   new_window_mock.ExpectAnyNavigations();
   server_mock_.ExpectAndServeAnyRequests(GetParam());
 
   // Tell the page to open the popup. Some versions of IE will prevent a popup
   // unless a click is involved.
   EXPECT_CALL(ie_mock_, OnLoad(GetParam().invokes_cf(), StrEq(parent_url)));
@@ skipping 67 matching lines @@
     testing::Values(NavigationTransitionTestParameter(
         CFInvocation::META_TAG, CFInvocation::META_TAG)));
 INSTANTIATE_TEST_CASE_P(
     CFToIE,
     NavigationTransitionTest,
     testing::Values(NavigationTransitionTestParameter(
         CFInvocation::META_TAG, CFInvocation::NONE)));
 
 // Test window.open calls.
 TEST_P(NavigationTransitionTest, JavascriptWindowOpen) {
+  // Please see http://code.google.com/p/chromium/issues/detail?id=60987
+  // for more information on why this test is disabled for Vista with IE7.
+  if (base::win::GetVersion() == base::win::VERSION_VISTA &&
+      GetInstalledIEVersion() == IE_7) {
+    LOG(INFO) << "Not running test on Vista with IE7";
+    return;
+  }
+
   std::wstring parent_url = GetTestUrl(L"window_open.html?simple.html");
   std::wstring new_window_url = GetSimplePageUrl();
   MockAccEventObserver acc_observer;
   testing::StrictMock<MockIEEventSink> new_window_mock;
 
   ie_mock_.ExpectNavigation(page1_.invokes_cf(), parent_url);
   server_mock_.ExpectAndServeRequest(page1_, parent_url);
   EXPECT_CALL(ie_mock_, OnLoad(page1_.invokes_cf(), StrEq(parent_url)));
   // Tell the page to open the popup. Some versions of IE will prevent a popup
   // unless a click is involved.
@@ skipping 254 matching lines @@
 // See bug http://crbug.com/36694
 // This test does the following:-
 // Navigates IE to a URL which in ChromeFrame.
 // Performs a top level form post in the document
 // In response to the POST we send over an attachment via the
 // content-disposition header.
 // IE brings up a file open dialog in this context.
 // We bring up the Save dialog via accessibility and save the file
 // and validate that all is well.
 TEST_F(FullTabDownloadTest, CF_DownloadFileFromPost) {
+  // Please see http://code.google.com/p/chromium/issues/detail?id=60987
+  // for more information on why this test is disabled for Vista with IE7.
+  if (base::win::GetVersion() == base::win::VERSION_VISTA &&
+      GetInstalledIEVersion() == IE_7) {
+    LOG(INFO) << "Not running test on Vista with IE7";
+    return;
+  }
   chrome_frame_test::MockWindowObserver download_watcher;
   download_watcher.WatchWindow("File Download", "");
 
   chrome_frame_test::MockWindowObserver save_dialog_watcher;
   save_dialog_watcher.WatchWindow("Save As", "");
 
   EXPECT_CALL(server_mock_, Get(_, StrEq(L"/post_source.html"), _)).WillOnce(
     SendFast(
       "HTTP/1.1 200 OK\r\n"
       "Content-Type: text/html\r\n",
@@ skipping 180 matching lines @@
           DelayRefresh(&ie_mock_, &loop_, 2000),
           DelayCloseBrowserMock(&loop_, 4000, &ie_mock_)));
 
   EXPECT_CALL(ie_mock_, OnLoad(in_cf, StrEq(src_url)))
       .Times(2);
 
   LaunchIEAndNavigate(src_url);
 }
 
 }  // namespace chrome_frame_test