[GTTF] Various ui tests cleanups:

chrome/browser/extensions/extension_uitest.cc

 // Copyright (c) 2009 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 "base/command_line.h"
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/values.h"
 #include "chrome/browser/automation/extension_automation_constants.h"
 #include "chrome/browser/extensions/extension_tabs_module_constants.h"
@@ skipping 265 matching lines @@
             &ExtensionTestRoundtripApiCall::CheckAndSendResponse))));
 
   EXPECT_CALL(*mock_, HandleClosed(_));
 
   ASSERT_EQ(AUTOMATION_MSG_NAVIGATION_SUCCESS, tab_->NavigateInExternalTab(
     GURL("chrome-extension://ofoknjclcmghjfmbncljcnpjmfmldhno/test.html"),
     GURL("")));
 
   // CheckAndSendResponse (called by OnForwardMessageToExternalHost)
   // will end the loop once it has received both of our expected messages.
-  loop_.RunFor(2 * action_max_timeout_ms());
+  loop_.RunFor(action_max_timeout_ms());
 }
 
 class ExtensionTestBrowserEvents : public ExtensionUITest {
  public:
   ExtensionTestBrowserEvents()
     : ExtensionUITest(kTestDirectoryBrowserEvent),
       response_count_(0) {
   }
 
   void SetUp() {
@@ skipping 166 matching lines @@
             &ExtensionTestBrowserEvents::HandleMessageFromChrome))));
 
   EXPECT_CALL(*mock_, HandleClosed(_));
 
   ASSERT_EQ(AUTOMATION_MSG_NAVIGATION_SUCCESS, tab_->NavigateInExternalTab(
     GURL("chrome-extension://ofoknjclcmghjfmbncljcnpjmfmldhno/test.html"),
     GURL("")));
 
   // HandleMessageFromChrome (called by OnForwardMessageToExternalHost) ends
   // the loop when we've received the number of response messages we expect.
-  loop_.RunFor(2 * action_max_timeout_ms());
+  loop_.RunFor(action_max_timeout_ms());
 
   // If this assert hits and the actual size is 0 then you need to look at:
   // src\chrome\test\data\extensions\uitest\event_sink\test.html and see if
   // all the events we are attaching to are valid. Also compare the list against
   // the event_names_ string array above.
   ASSERT_EQ(arraysize(events_), event_count_.size());
   for (std::map<std::string, int>::iterator i = event_count_.begin();
       i != event_count_.end(); ++i) {
     const std::pair<std::string, int>& value = *i;
     EXPECT_EQ(1, value.second);
   }
 }
 #endif  // defined(OS_WIN)
 
 }  // namespace