Session/tab sync performance tests.

chrome/test/live_sync/live_sessions_sync_test.cc

 // Copyright (c) 2011 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 "chrome/test/live_sync/live_sessions_sync_test.h"
 
 #include "base/test/test_timeouts.h"
 #include "base/time.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sync/profile_sync_service.h"
@@ skipping 71 matching lines @@
         }
         return true;
       }
     }
   }
   VLOG(1) << "Could not find tab with url " << url.spec();
   return false;
 }
 
 bool LiveSessionsSyncTest::OpenTab(int index, const GURL& url) {
-  static const int timeout_milli = TestTimeouts::action_max_timeout_ms();
   VLOG(1) << "Opening tab: " << url.spec();
   GetBrowser(index)->ShowSingletonTab(url);
+  return AwaitSessionPropagation(index, std::vector<GURL>(1, url));
+}
+
+bool LiveSessionsSyncTest::OpenMultipleTabs(int index,
+                                            const std::vector<GURL>& urls) {
+  Browser* browser = GetBrowser(index);
+  for (std::vector<GURL>::const_iterator it = urls.begin();
+       it != urls.end(); ++it) {
+    VLOG(1) << "Opening tab: " << it->spec();
+    browser->ShowSingletonTab(*it);
+  }
+  return AwaitSessionPropagation(index, urls);
+}
+
+bool LiveSessionsSyncTest::AwaitSessionPropagation(
+    int index, const std::vector<GURL>& urls) {
   VLOG(1) << "Waiting for session to propagate to associator.";
+  static const int timeout_milli = TestTimeouts::action_max_timeout_ms();
   base::TimeTicks start_time = base::TimeTicks::Now();
   base::TimeTicks end_time = start_time +
                              base::TimeDelta::FromMilliseconds(timeout_milli);
-  do {
-    if (ModelAssociatorHasTabWithUrl(index, url))
-      return true;
-    GetProfile(index)->GetProfileSyncService()->GetSessionModelAssociator()->
-        BlockUntilLocalChangeForTest(timeout_milli);
-    ui_test_utils::RunMessageLoop();
-  } while (base::TimeTicks::Now() < end_time);
-
-  if (ModelAssociatorHasTabWithUrl(index, url))
-    return true;
-
-  LOG(ERROR) << "Failed to find tab after " << timeout_milli/1000.0
-             << " seconds.";
-  return false;
+  bool found;
+  for (std::vector<GURL>::const_iterator it = urls.begin();
+       it != urls.end(); ++it) {
+    found = false;

[braffert, 2011/08/04 23:47:34]

Using a flag here to avoid calling ModelAssociatorHasTabWithUrl twice.

+    do {
+      if (base::TimeTicks::Now() >= end_time) {
+        LOG(ERROR) << "Failed to find all tabs after " << timeout_milli/1000.0
+                   << " seconds.";
+        return false;
+      }
+      found = ModelAssociatorHasTabWithUrl(index, *it);

[Nicolas Zea, 2011/08/05 00:40:02]

May as well put this before the timeout check.

[braffert, 2011/08/05 00:47:38]

I'm not seeing why one way is better than the other, but done.

+      if (!found) {
+        GetProfile(index)->GetProfileSyncService()->
+            GetSessionModelAssociator()->
+            BlockUntilLocalChangeForTest(timeout_milli);
+        ui_test_utils::RunMessageLoop();
+      }
+    } while (!found);
+  }
+  return true;
 }
 
 std::vector<SessionWindow*>* LiveSessionsSyncTest::GetLocalWindows(int index) {
   // The local session provided by GetLocalSession is owned, and has lifetime
   // controlled, by the model associator, so we must make our own copy.
   const SyncedSession* local_session;
   if (!GetLocalSession(index, &local_session)) {
     return NULL;
   }
   scoped_ptr<SyncedSession> session_copy(new SyncedSession());
@@ skipping 181 matching lines @@
   // sync profile.
   BrowserList::CloseAllBrowsers();
   ui_test_utils::RunAllPendingInMessageLoop();
 
   // All browsers should be closed at this point, else when the framework
   // calls QuitBrowsers() we could see memory corruption.
   ASSERT_EQ(0U, BrowserList::size());
 
   LiveSyncTest::CleanUpOnMainThread();
 }