Fix testLitePageFallbackViaPagePolicies below M61

Fix testLitePageFallbackViaPagePolicies below M61

BUG=732606
Review-Url: https://codereview.chromium.org/2939943003
Cr-Commit-Position: refs/heads/master@{#479818}
Committed: https://chromium.googlesource.com/chromium/src/+/455f6d8c01ce70db7f311a0308e943fc7b612358

[Robert Ogden, 2017-06-14 21:43:16]

robertogden@chromium.org changed reviewers:
+ ryansturm@chromium.org

[Robert Ogden, 2017-06-14 21:43:17]

Ryan, please take a look

[RyanSturm, 2017-06-14 21:49:59]

https://codereview.chromium.org/2939943003/diff/1/tools/chrome_proxy/webdrive...
File tools/chrome_proxy/webdriver/lite_page.py (right):

https://codereview.chromium.org/2939943003/diff/1/tools/chrome_proxy/webdrive...
tools/chrome_proxy/webdriver/lite_page.py:172: @ChromeVersionAfterM(61)
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:21]

https://codereview.chromium.org/2939943003/diff/1/tools/chrome_proxy/webdrive...
File tools/chrome_proxy/webdriver/lite_page.py (right):

https://codereview.chromium.org/2939943003/diff/1/tools/chrome_proxy/webdrive...
tools/chrome_proxy/webdriver/lite_page.py:172: @ChromeVersionAfterM(61)
On 2017/06/14 21:49:58, RyanSturm wrote:
> 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.

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]

https://codereview.chromium.org/2939943003/diff/1/tools/chrome_proxy/webdrive...
File tools/chrome_proxy/webdriver/lite_page.py (right):

https://codereview.chromium.org/2939943003/diff/1/tools/chrome_proxy/webdrive...
tools/chrome_proxy/webdriver/lite_page.py:172: @ChromeVersionAfterM(61)
On 2017/06/14 21:53:20, Robert Ogden wrote:
> On 2017/06/14 21:49:58, RyanSturm wrote:
> > 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.
> 
> This will check if version >= 61 and run if true. See
>
https://cs.chromium.org/chromium/src/tools/chrome_proxy/webdriver/decorators....

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]

https://codereview.chromium.org/2939943003/diff/1/tools/chrome_proxy/webdrive...
File tools/chrome_proxy/webdriver/lite_page.py (right):

https://codereview.chromium.org/2939943003/diff/1/tools/chrome_proxy/webdrive...
tools/chrome_proxy/webdriver/lite_page.py:172: @ChromeVersionAfterM(61)
On 2017/06/14 21:55:38, RyanSturm wrote:
> On 2017/06/14 21:53:20, Robert Ogden wrote:
> > On 2017/06/14 21:49:58, RyanSturm wrote:
> > > 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.
> > 
> > This will check if version >= 61 and run if true. See
> >
>
https://cs.chromium.org/chromium/src/tools/chrome_proxy/webdriver/decorators....
> 
> It's a pretty bad decorator name then :/ Maybe you should adjust the >= in the
> decorator to > and move this to 60?

Changed name to make it clearer

[RyanSturm, 2017-06-15 20:27:18]

lgtm

[Robert Ogden, 2017-06-15 20:29:45]

The CQ bit was checked by robertogden@chromium.org

[commit-bot: I haz the power, 2017-06-15 20:30:11]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-15 20:41:08]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1497558585052090,
"parent_rev": "4ab7c29b4c7ad38decdacfe842d8986886ee70cd", "commit_rev":
"455f6d8c01ce70db7f311a0308e943fc7b612358"}

[commit-bot: I haz the power, 2017-06-15 20:41:17]

Description was changed from

==========
Fix testLitePageFallbackViaPagePolicies below M61

BUG=732606
==========

to

==========
Fix testLitePageFallbackViaPagePolicies below M61

BUG=732606

Review-Url: https://codereview.chromium.org/2939943003
Cr-Commit-Position: refs/heads/master@{#479818}
Committed:
https://chromium.googlesource.com/chromium/src/+/455f6d8c01ce70db7f311a0308e9...
==========

[commit-bot: I haz the power, 2017-06-15 20:41:18]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/455f6d8c01ce70db7f311a0308e9...