Get rid of WaitForWindowCountToChange.

The function had confusing semantics, was marked to be removed,
and could induce flakiness in tests which used it.

WaitForWindowCountToBecome is much better replacement for it.

Review URL: http://codereview.chromium.org/49007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@12348 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 0 insertions, 14 deletions

diff --git a/chrome/test/automation/automation_proxy.cc b/chrome/test/automation/automation_proxy.cc
index 35d44f8..3a8f406 100644
--- a/chrome/test/automation/automation_proxy.cc
+++ b/chrome/test/automation/automation_proxy.cc
@@ -251,20 +251,6 @@ bool AutomationProxy::GetBrowserWindowCount(int* num_windows) {
   return succeeded;
 }
 
-bool AutomationProxy::WaitForWindowCountToChange(int count, int* new_count,
-                                                 int wait_timeout) {
-  const TimeTicks start = TimeTicks::Now();
-  const TimeDelta timeout = TimeDelta::FromMilliseconds(wait_timeout);
-  while (TimeTicks::Now() - start < timeout) {
-    bool succeeded = GetBrowserWindowCount(new_count);
-    if (!succeeded) return false;
-    if (count != *new_count) return true;
-    PlatformThread::Sleep(automation::kSleepTime);
-  }
-  // Window count never changed.
-  return false;
-}
-
 bool AutomationProxy::WaitForWindowCountToBecome(int count,
                                                  int wait_timeout) {
   const TimeTicks start = TimeTicks::Now();