Remove page_measurement_results.

tools/perf/metrics/test_page_measurement_results.py

Index: tools/perf/metrics/test_page_measurement_results.py
diff --git a/tools/perf/metrics/test_page_measurement_results.py b/tools/perf/metrics/test_page_measurement_results.py
index 8d17e5b575771b8f6b1f404144869e582a967f2f..22774110be782b90747031f2dd27d8b1fb177763 100644
--- a/tools/perf/metrics/test_page_measurement_results.py
+++ b/tools/perf/metrics/test_page_measurement_results.py
@@ -3,14 +3,14 @@
 # found in the LICENSE file.
 
 from telemetry.page import page as page_module
-from telemetry.results import page_measurement_results
+from telemetry.results import page_test_results
 from telemetry.value import scalar
 
 
-class TestPageMeasurementResults(
-    page_measurement_results.PageMeasurementResults):
+class TestPageTestResults(

[chrishenry, 2014/07/31 05:01:12]

Can we kill this class instead? Pretty please? Let's just fix the users to check
for the values directly.

[nednguyen, 2014/07/31 14:24:51]

I can do it a different patch. There maybe some the common pattern of testing
results that appear often enough that people need at least utility functions to
compare results.

[chrishenry, 2014/07/31 15:24:27]

There might be indeed, but it shouldn't need a subclass of PageTestResults
(perhaps a subclass of unittest or just a free function).

That said, the assertion below is poorly implemented anyway.

+    page_test_results.PageTestResults):
   def __init__(self, test):
-    super(TestPageMeasurementResults, self).__init__()
+    super(TestPageTestResults, self).__init__()
     self.test = test
     page = page_module.Page("http://www.google.com", {})
     self.StartTest(page)