Fix testLitePageFallbackViaPagePolicies below M61

tools/chrome_proxy/webdriver/lite_page.py

 # Copyright 2017 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.
 
 import common
 from common import TestDriver
 from common import IntegrationTest
+from decorators import ChromeVersionAfterM
 
 import time
 
 class LitePage(IntegrationTest):
 
   # Checks that a Lite Page is served and the force_lite_page experiment
   # directive is provided when always-on.
   def testLitePageForcedExperiment(self):
     # If it was attempted to run with another experiment, skip this test.
     if common.ParseFlags().browser_args and ('--data-reduction-proxy-experiment'
@@ skipping 143 matching lines @@
 
         # Lo-Fi fallback is not currently supported via the client. Check that
         # no Lo-Fi response is received.
         self.checkLoFiResponse(response, False)
 
       # Verify that a Lite Page was requested.
       self.assertEqual(1, lite_page_requests)
 
   # Verifies Lo-Fi fallback via the page-policies server directive.
   # Note: this test is for the CPAT protocol change in M-61.
+  @ChromeVersionAfterM(61)

[RyanSturm, 2017/06/14 21:49:58]

Tricky question if this should be 61 or 60, since it will be done by the end of
61, and we want to test 61 stable with this.

[Robert Ogden, 2017/06/14 21:53:20]

This will check if version >= 61 and run if true. See
https://cs.chromium.org/chromium/src/tools/chrome_proxy/webdriver/decorators....

[RyanSturm, 2017/06/14 21:55:38]

It's a pretty bad decorator name then :/ Maybe you should adjust the >= in the
decorator to > and move this to 60?

[Robert Ogden, 2017/06/14 22:33:45]

Changed name to make it clearer

   def testLitePageFallbackViaPagePolicies(self):
     with TestDriver() as test_driver:
       test_driver.AddChromeArg('--enable-spdy-proxy-auth')
       test_driver.AddChromeArg('--enable-features='
                                'DataReductionProxyDecidesTransform')
       test_driver.AddChromeArg('--force-fieldtrial-params='
                                'NetworkQualityEstimator.Enabled:'
                                'force_effective_connection_type/Slow2G,'
                                'DataCompressionProxyLoFi.Enabled_Preview:'
                                'effective_connection_type/2G')
@@ skipping 89 matching lines @@
           if 'chrome-proxy' in response.response_headers:
             self.assertNotIn('page-policies',
                              response.response_headers['chrome-proxy'])
         else:
           # No subresources should accept transforms.
           self.assertNotIn('chrome-proxy-accept-transform',
             response.request_headers)
 
 if __name__ == '__main__':
   IntegrationTest.RunAllTests()