Add a method for setting the value of an Accessibility object....

chrome_frame/test/ui_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 <mshtmcid.h>
 #include <string>
 
 #include "base/scoped_variant_win.h"
+#include "base/test/test_file_util.h"
 #include "chrome/common/url_constants.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"
 
 #include "testing/gmock_mutant.h"
 
 using testing::_;
 using testing::InSequence;
@@ skipping 417 matching lines @@
   EXPECT_CALL(win_observer_mock,
               OnWindowDetected(_, testing::StartsWith(kPageInfoCaption)))
       .WillOnce(testing::DoAll(
           DelayDoCloseWindow(5000),  // wait to catch possible crash
           DelayCloseBrowserMock(&loop_, 5500, &ie_mock_)));
 
   LaunchIENavigateAndLoop(GetSimplePageUrl(),
                           kChromeFrameLongNavigationTimeoutInSeconds * 2);
 }
 
-TEST_F(ContextMenuTest, FLAKY_CFSaveAs) {
+TEST_F(ContextMenuTest, CFSaveAs) {
   server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag());
   MockWindowObserver win_observer_mock;
   InSequence expect_in_sequence_for_scope;
 
   // Open 'Save As' dialog.
   const wchar_t* kSaveDlgClass = L"#32770";
   EXPECT_CALL(acc_observer_, OnAccDocLoad(_))
       .WillOnce(testing::DoAll(
           WatchWindow(&win_observer_mock, kSaveDlgClass),
           OpenContextMenuAsync()));
   EXPECT_CALL(acc_observer_, OnMenuPopup(_))
       .WillOnce(DoDefaultAction(AccObjectMatcher(L"Save as...")));
 
+  // Get safe download name using temporary file.
   FilePath temp_file_path;
-  EXPECT_TRUE(file_util::CreateTemporaryFile(&temp_file_path));
+  ASSERT_TRUE(file_util::CreateTemporaryFile(&temp_file_path));
   temp_file_path = temp_file_path.ReplaceExtension(L".htm");
+  ASSERT_TRUE(file_util::DieFileDie(temp_file_path, false));
 
-  const wchar_t* kSaveFileName = temp_file_path.value().c_str();
-  DeleteFile(kSaveFileName);
-
-  const char* kSaveDlgCaption = "Save As";
-  EXPECT_CALL(win_observer_mock, OnWindowDetected(_, StrEq(kSaveDlgCaption)))
+  EXPECT_CALL(win_observer_mock, OnWindowDetected(_, StrEq("Save As")))
       .WillOnce(testing::DoAll(
-          DelaySendString(&loop_, 100, kSaveFileName),
-          DelaySendChar(&loop_, 200, VK_RETURN, simulate_input::NONE),
-          DelayCloseBrowserMock(&loop_, 4000, &ie_mock_)));
+          SetAccObjectValue(AccObjectMatcher(L"File name:", L"", L"simple*"),
+                            temp_file_path.value()),
+          DoDefaultAction(AccObjectMatcher(L"Save", L"push button")),
+          CloseWhenFileSaved(&ie_mock_, temp_file_path, 5000)));
 
   LaunchIENavigateAndLoop(GetSimplePageUrl(),
                           kChromeFrameLongNavigationTimeoutInSeconds * 2);
-  ASSERT_NE(INVALID_FILE_ATTRIBUTES, GetFileAttributes(kSaveFileName));
-  ASSERT_TRUE(DeleteFile(kSaveFileName));
+  ASSERT_TRUE(file_util::DieFileDie(temp_file_path, false));
 }
 
 // This tests that the about:version page can be opened via the CF context menu.
 TEST_F(ContextMenuTest, CFAboutVersionLoads) {
   server_mock_.ExpectAndServeAnyRequests(CFInvocation::MetaTag());
   const wchar_t* kAboutVersionUrl = L"gcf:about:version";
   const wchar_t* kAboutVersionWithoutProtoUrl = L"about:version";
   MockIEEventSink new_window_mock;
   new_window_mock.ExpectAnyNavigations();
   InSequence expect_in_sequence_for_scope;
@@ skipping 92 matching lines @@
   EXPECT_CALL(acc_observer_, OnMenuPopup(_))
       .WillOnce(DoDefaultAction(AccObjectMatcher(L"Forward")));
 
   EXPECT_CALL(ie_mock_, OnLoad(IN_IE, StrEq(page2)))
       .WillOnce(CloseBrowserMock(&ie_mock_));
 
   LaunchIEAndNavigate(page1);
 }
 
 }  // namespace chrome_frame_test