Adds lite page integration test (ignoring country blacklist).

Adds lite page integration test (ignoring country blacklist).
Also:
  - removes no longer needed field trial on some lite page
integration tests
  - adds server response checks on new force lofi flag test
  - annotates the tests than need M-61 or greater.

BUG=701802
Review-Url: https://codereview.chromium.org/2941103002
Cr-Commit-Position: refs/heads/master@{#480861}
Committed: https://chromium.googlesource.com/chromium/src/+/b8ae3bdd6f625ef983b2615b74f240506dcbde8b

[dougarnett, 2017-06-15 19:27:58]

The CQ bit was checked by dougarnett@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-15 19:28:30]

Dry run: CQ is trying da patch.

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

[dougarnett, 2017-06-15 19:32:54]

dougarnett@chromium.org changed reviewers:
+ bustamante@chromium.org, megjablon@chromium.org

[dougarnett, 2017-06-15 19:32:57]

Resolved with buettner that we will retain the exp=ignore_preview_blacklist in
order to be able to test main path lite pages in intergration tests that may run
in copyright restricted country (like the US).
This CL adds a test for that (plus some cleanup on couple other tests).

[bustamante, 2017-06-15 19:50:28]

On 2017/06/15 19:32:57, dougarnett wrote:
> Resolved with buettner that we will retain the exp=ignore_preview_blacklist in
> order to be able to test main path lite pages in intergration tests that may
run
> in copyright restricted country (like the US).
> This CL adds a test for that (plus some cleanup on couple other tests).

lgtm, but the decorator is getting changed in a pending CL
https://codereview.chromium.org/2939943003/

[dougarnett, 2017-06-15 22:38:42]

On 2017/06/15 19:50:28, bustamante wrote:
> On 2017/06/15 19:32:57, dougarnett wrote:
> > Resolved with buettner that we will retain the exp=ignore_preview_blacklist
in
> > order to be able to test main path lite pages in intergration tests that may
> run
> > in copyright restricted country (like the US).
> > This CL adds a test for that (plus some cleanup on couple other tests).
> 
> lgtm, but the decorator is getting changed in a pending CL
> https://codereview.chromium.org/2939943003/

Thanks, merged with decorator name change.

[dougarnett, 2017-06-16 20:46:29]

Description was changed from

==========
Adds lite page integration test (ignoring country blacklist).
Also removes no longer needed field trial on some lite page
integration tests and annotates the tests than need M-61 or
greater.

BUG=701802
==========

to

==========
Adds lite page integration test (ignoring country blacklist).
Also:
  - removes no longer needed field trial on some lite page
integration tests
  - adds server response checks on new force lofi flag test
  - annotates the tests than need M-61 or greater.

BUG=701802
==========

[dougarnett, 2017-06-19 15:57:28]

Ping Megan - do you want to look at these integration test improvements?

[megjablon, 2017-06-19 20:14:48]

lgtm

[dougarnett, 2017-06-19 22:28:04]

The CQ bit was checked by dougarnett@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-19 22:28:45]

Dry run: CQ is trying da patch.

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

[dougarnett, 2017-06-19 22:30:34]

Renamed two triggering tests (Slow/Fast) from testLitePage* to testPreview* to
better match allowed server behavior.

Richard, do you want to take another looks since I made a few changes since your
original review?

[commit-bot: I haz the power, 2017-06-19 23:46:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-19 23:46:09]

Dry run: This issue passed the CQ dry run.

[dougarnett, 2017-06-20 16:47:32]

The CQ bit was checked by dougarnett@chromium.org

[dougarnett, 2017-06-20 16:47:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
bustamante@chromium.org, megjablon@chromium.org
Link to the patchset: https://codereview.chromium.org/2941103002/#ps80001
(title: "Renames two triggering tests to be less specific wrt actual previews
server selects.")

[commit-bot: I haz the power, 2017-06-20 16:47:49]

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-20 16:54:59]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1497977252915910,
"parent_rev": "9a57c3c2613c24086d2c44f4718edf501cf067cc", "commit_rev":
"b8ae3bdd6f625ef983b2615b74f240506dcbde8b"}

[commit-bot: I haz the power, 2017-06-20 16:55:14]

Description was changed from

==========
Adds lite page integration test (ignoring country blacklist).
Also:
  - removes no longer needed field trial on some lite page
integration tests
  - adds server response checks on new force lofi flag test
  - annotates the tests than need M-61 or greater.

BUG=701802
==========

to

==========
Adds lite page integration test (ignoring country blacklist).
Also:
  - removes no longer needed field trial on some lite page
integration tests
  - adds server response checks on new force lofi flag test
  - annotates the tests than need M-61 or greater.

BUG=701802

Review-Url: https://codereview.chromium.org/2941103002
Cr-Commit-Position: refs/heads/master@{#480861}
Committed:
https://chromium.googlesource.com/chromium/src/+/b8ae3bdd6f625ef983b2615b74f2...
==========

[commit-bot: I haz the power, 2017-06-20 16:55:17]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/b8ae3bdd6f625ef983b2615b74f2...

[bustamante, 2017-06-20 16:57:32]

lgtm