Kill page_test.NavigateToPage

tools/chrome_proxy/common/chrome_proxy_measurements.py

Index: tools/chrome_proxy/common/chrome_proxy_measurements.py
diff --git a/tools/chrome_proxy/common/chrome_proxy_measurements.py b/tools/chrome_proxy/common/chrome_proxy_measurements.py
index ab8fd671a5e8897becdf4d91d088673dc910de3a..eb42e0a2c1c2059263d7ec4b3c1153981f7fddfa 100644
--- a/tools/chrome_proxy/common/chrome_proxy_measurements.py
+++ b/tools/chrome_proxy/common/chrome_proxy_measurements.py
@@ -50,10 +50,6 @@ class ChromeProxyValidation(page_test.PageTest):
       return True
     return False
 
-  def RunNavigateSteps(self, page, tab):
-    # The redirect from safebrowsing causes a timeout. Ignore that.
-    try:
-      super(ChromeProxyValidation, self).RunNavigateSteps(page, tab)
       if self._expect_timeout:

[sclittle, 2015/05/19 19:58:57]

This is syntactically incorrect without the "try", and the method would have
already returned by this point (see the return statements above).

What are you trying to do here?

[nednguyen, 2015/05/19 20:18:01]

Err, silly me. I am about to remove the whole RunNavigateSteps method.

         raise metrics.ChromeProxyMetricException, (
             'Timeout was expected, but did not occur')