Merge all logic in PageMeasurementResults to PageTestResults.

tools/perf/measurements/page_cycler_unittest.py

Index: tools/perf/measurements/page_cycler_unittest.py
diff --git a/tools/perf/measurements/page_cycler_unittest.py b/tools/perf/measurements/page_cycler_unittest.py
index 414ed273c48219b43290798cf1d9f1f33acc30c0..0fd3169ab56c0d28cfc7b94fb06f75a58f4a961e 100644
--- a/tools/perf/measurements/page_cycler_unittest.py
+++ b/tools/perf/measurements/page_cycler_unittest.py
@@ -140,19 +140,18 @@ class PageCyclerUnitTest(unittest.TestCase):
     url_name = 'http://fakepage.com'
     page = FakePage(url_name)
     tab = FakeTab()
-    results = page_measurement_results.PageMeasurementResults()
 
     for i in range(5):
+      results = page_measurement_results.PageMeasurementResults()
+      results.StartTest(page)
       cycler.WillNavigateToPage(page, tab)
       self.assertEqual(max(0, i - 2), tab.clear_cache_calls,
                        'Iteration %d tab.clear_cache_calls %d' %
                        (i, tab.clear_cache_calls))
-      results.WillMeasurePage(page)
       cycler.MeasurePage(page, tab, results)
+      results.StopTest(page)
 
-      values = results.page_specific_values_for_current_page
-      results.DidMeasurePage()
-
+      values = results.all_page_specific_values
       self.assertGreater(len(values), 2)
 
       self.assertEqual(values[0].page, page)
@@ -166,16 +165,15 @@ class PageCyclerUnitTest(unittest.TestCase):
     cycler = self.SetUpCycler(['--pageset-repeat=3'], True)
     pages = [FakePage('http://fakepage1.com'), FakePage('http://fakepage2.com')]
     tab = FakeTab()
-    results = page_measurement_results.PageMeasurementResults()
     for i in range(3):
       for page in pages:
+        results = page_measurement_results.PageMeasurementResults()
+        results.StartTest(page)

[nednguyen, 2014/07/25 16:14:45]

I think we use one results throughout the whole telemetry test run, isn't it?
If so, a better test maybe asserting that the results contain all the values
across 3 runs.

[chrishenry, 2014/07/25 16:24:28]

I think that will make the test unnecessarily complicated. Another way to do
this is to check all_page_specific_values after every MeasurePage and check for
the additional values, but then that would be no better than this logic. s:

[nednguyen, 2014/07/25 16:36:57]

Ok. Chatted offline, we probably should fix the page_cylcer WillMeasure method
later.

         cycler.WillNavigateToPage(page, tab)
-        results.WillMeasurePage(page)
         cycler.MeasurePage(page, tab, results)
+        results.StopTest(page)
 
-        values = results.page_specific_values_for_current_page
-        results.DidMeasurePage()
-
+        values = results.all_page_specific_values
         self.assertGreater(len(values), 2)
 
         self.assertEqual(values[0].page, page)
@@ -191,17 +189,16 @@ class PageCyclerUnitTest(unittest.TestCase):
 
     pages = [FakePage('http://fakepage1.com'), FakePage('http://fakepage2.com')]
     tab = FakeTab()
-    results = page_measurement_results.PageMeasurementResults()
 
     for i in range(2):
       for page in pages:
+        results = page_measurement_results.PageMeasurementResults()
+        results.StartTest(page)
         cycler.WillNavigateToPage(page, tab)
-        results.WillMeasurePage(page)
         cycler.MeasurePage(page, tab, results)
+        results.StopTest(page)
 
-        values = results.page_specific_values_for_current_page
-        results.DidMeasurePage()
-
+        values = results.all_page_specific_values
         self.assertEqual(4, len(values))
 
         self.assertEqual(values[0].page, page)
@@ -215,5 +212,4 @@ class PageCyclerUnitTest(unittest.TestCase):
                            'cpu_utilization.cpu_utilization_%s' % expected)
           self.assertEqual(value.units, '%')
 
-
         cycler.DidNavigateToPage(page, tab)