[telemetry] Re-enable testGotQueryParams.

tools/telemetry/telemetry/page/page_measurement_unittest.py

Index: tools/telemetry/telemetry/page/page_measurement_unittest.py
diff --git a/tools/telemetry/telemetry/page/page_measurement_unittest.py b/tools/telemetry/telemetry/page/page_measurement_unittest.py
index 32282636119e0a33e44bb8ea0d2c16499d0826c2..01fdb0f3b0610213f897200444b0fb3f0ab54ec1 100644
--- a/tools/telemetry/telemetry/page/page_measurement_unittest.py
+++ b/tools/telemetry/telemetry/page/page_measurement_unittest.py
@@ -23,13 +23,19 @@ class MeasurementThatHasDefaults(page_measurement.PageMeasurement):
     parser.add_option('-x', dest='x', default=3)
 
   def MeasurePage(self, page, tab, results):
-    assert self.options.x == 3
+    if not hasattr(self.options, 'x'):
+      raise page_measurement.MeasurementFailure('Default option was not set.')
+    if self.options.x != 3:
+      raise page_measurement.MeasurementFailure(
+          'Expected x == 3, got x == ' + self.options.x)
     results.Add('x', 'ms', 7)
 
 class MeasurementForBlank(page_measurement.PageMeasurement):
   def MeasurePage(self, page, tab, results):
     contents = tab.EvaluateJavaScript('document.body.textContent')
-    assert contents.strip() == 'Hello world'
+    if contents.strip() != 'Hello world':
+      raise page_measurement.MeasurementFailure(
+          'Page contents were: ' + contents)
 
 class MeasurementForReplay(page_measurement.PageMeasurement):
   def MeasurePage(self, page, tab, results):
@@ -41,7 +47,10 @@ class MeasurementForReplay(page_measurement.PageMeasurement):
 class MeasurementQueryParams(page_measurement.PageMeasurement):
   def MeasurePage(self, page, tab, results):
     query = tab.EvaluateJavaScript('window.location.search')
-    assert query.strip() == '?foo=1'
+    expected = '?foo=1'
+    if query.strip() != expected:
+      raise page_measurement.MeasurementFailure(
+          'query was %s, not %s.' % (query, expected))
 
 class MeasurementWithAction(page_measurement.PageMeasurement):
   def __init__(self):
@@ -63,11 +72,9 @@ class PageMeasurementUnitTest(
     all_results = self.RunMeasurement(measurement, ps, options=self._options)
     self.assertEquals(0, len(all_results.failures))
 
-  def disabled_testGotQueryParams(self):
-    # Disabled due to http://crbug.com/288631
+  def testGotQueryParams(self):
     ps = self.CreatePageSet('file://blank.html?foo=1')
     measurement = MeasurementQueryParams()
-    ps.pages[-1].query_params = '?foo=1'
     all_results = self.RunMeasurement(measurement, ps, options=self._options)
     self.assertEquals(0, len(all_results.failures))