[Android] Don't wait for background tabs

tools/perf/measurements/session_restore.py

 # Copyright 2013 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.
 
 from measurements import startup
 from metrics import cpu
 from metrics import startup_metric
 
 
 class SessionRestore(startup.Startup):
@@ skipping 32 matching lines @@
 
     if len(wpr_archives.keys()) > 1:
       raise Exception("Invalid pageset: more than 1 WPR archive found.: " +
           ', '.join(wpr_archives.keys()))
 
   def DidStartBrowser(self, browser):
     self._cpu_metric = cpu.CpuMetric(browser)
     self._cpu_metric.Start(None, None)
 
   def MeasurePage(self, page, tab, results):
-    # Wait for all tabs to finish loading.
-    for i in xrange(len(tab.browser.tabs)):
-      t = tab.browser.tabs[i]
-      t.WaitForDocumentReadyStateToBeComplete()
-
+    # Wait for foreground tab to finish loading. We can't wait for all tabs to
+    # finish loading because on Android the data of the background tabs is
+    # loaded on demand, when the user switches to them, rather than during
+    # startup.
+    # We are assuming that the foreground tab is tab[0]. This is true in current
+    # code, but is not guaranteed by the APIs.
+    t = tab.browser.tabs[0]

[jeremy, 2014/02/13 13:35:26]

Looks like we have no way of checking that this is in fact the frontmost tab
which kind of scares me :/

[aberent, 2014/02/13 15:59:30]

Fixed, I think (just testing) although it only works through luck on Android
(due to a bug). See the next patch.

+    t.WaitForDocumentReadyStateToBeComplete()
     # Record CPU usage from browser start to when all pages have loaded.
     self._cpu_metric.Stop(None, None)
     self._cpu_metric.AddResults(tab, results, 'cpu_utilization')
 
     startup_metric.StartupMetric().AddResults(tab, results)
 
     # TODO(jeremy): Measure time to load - first, last and frontmost tab here.