Start reporting watch time if ABR adapts above 200p.

Start reporting watch time if ABR adapts above 200p.

Previously we only checked the initial resolution and assumed it
was always large enough to start watch time reporting; clearly
this is not true since YouTube offers a 140p format.

Probably we also want to drop the minimum resolution to 200x140p,
which I'll check with product about and handle in a followup CL.

BUG=711792
TEST=new tests. Manual: force 140p, adapt above, verify watch time.

Review-Url: https://codereview.chromium.org/2822543006
Cr-Commit-Position: refs/heads/master@{#464826}
Committed: https://chromium.googlesource.com/chromium/src/+/2540556a43a467a11d4c2cffdb9457b97d8b94c4

[DaleCurtis, 2017-04-14 21:58:21]

dalecurtis@chromium.org changed reviewers:
+ sandersd@chromium.org

[sandersd (OOO until July 31), 2017-04-14 22:07:50]

lgtm

[DaleCurtis, 2017-04-14 22:12:53]

The CQ bit was checked by dalecurtis@chromium.org

[DaleCurtis, 2017-04-14 22:12:53]

Thanks for review! Simplified the logic a bit.

[DaleCurtis, 2017-04-14 22:12:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
sandersd@chromium.org
Link to the patchset: https://codereview.chromium.org/2822543006/#ps20001
(title: "Simplify")

[commit-bot: I haz the power, 2017-04-14 22:13:09]

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

[DaleCurtis, 2017-04-14 22:17:51]

Description was changed from

==========
Start reporting watch time if ABR adapts above 200p.

Previously we only checked the initial resolution and assumed it
was always large enough to start watch time reporting; clearly
this is not true since YouTube offers a 140p format.

Probably we also want to drop the minimum resolution to 200x140p,
which I'll check with product about and handle in a followup CL.

BUG=711792
TEST=manual, force 140p, adapt above, verify watch time.
==========

to

==========
Start reporting watch time if ABR adapts above 200p.

Previously we only checked the initial resolution and assumed it
was always large enough to start watch time reporting; clearly
this is not true since YouTube offers a 140p format.

Probably we also want to drop the minimum resolution to 200x140p,
which I'll check with product about and handle in a followup CL.

BUG=711792
TEST=new tests. Manual: force 140p, adapt above, verify watch time.
==========

[commit-bot: I haz the power, 2017-04-14 23:35:18]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492207973620860,
"parent_rev": "aec6e3f43c5681c6a67f8611284d13e6507c2a8f", "commit_rev":
"2540556a43a467a11d4c2cffdb9457b97d8b94c4"}

[commit-bot: I haz the power, 2017-04-14 23:36:06]

Description was changed from

==========
Start reporting watch time if ABR adapts above 200p.

Previously we only checked the initial resolution and assumed it
was always large enough to start watch time reporting; clearly
this is not true since YouTube offers a 140p format.

Probably we also want to drop the minimum resolution to 200x140p,
which I'll check with product about and handle in a followup CL.

BUG=711792
TEST=new tests. Manual: force 140p, adapt above, verify watch time.
==========

to

==========
Start reporting watch time if ABR adapts above 200p.

Previously we only checked the initial resolution and assumed it
was always large enough to start watch time reporting; clearly
this is not true since YouTube offers a 140p format.

Probably we also want to drop the minimum resolution to 200x140p,
which I'll check with product about and handle in a followup CL.

BUG=711792
TEST=new tests. Manual: force 140p, adapt above, verify watch time.

Review-Url: https://codereview.chromium.org/2822543006
Cr-Commit-Position: refs/heads/master@{#464826}
Committed:
https://chromium.googlesource.com/chromium/src/+/2540556a43a467a11d4c2cffdb94...
==========

[commit-bot: I haz the power, 2017-04-14 23:36:07]

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