Update selection of resolution policy for screen capture with getUserMedia.

Update selection of resolution policy for screen capture with getUserMedia.

The intent of the new algorithm is to allow resolution adjustment if the
resolution range allowed by constraints is a superset of the capabilities.
However, the actual capabilities are not currently known and, in practice,
resolution adjustment was allowed only when no constraints were specified.

This CL relaxes this rule to allow resolution adjustment by treating a max
constraint as if it were the actual capability. This should be updated so
that the actual native screen resolution is used as capability, but, in the
meantime, this fix is more compatible with existing applications that expect
the behavior of the old algorithm. Moreover, this patch does not affect spec
compliance since screen-resolution policies are not regulated by the spec as
long as the output resolution is within the constrained range.

BUG=709915
Review-Url: https://codereview.chromium.org/2800343005
Cr-Commit-Position: refs/heads/master@{#463309}
Committed: https://chromium.googlesource.com/chromium/src/+/71ffa0d2e073d09d8098b0794f7f382655aabedc

[Guido Urdaneta, 2017-04-10 15:45:30]

guidou@chromium.org changed reviewers:
+ hbos@chromium.org

[Guido Urdaneta, 2017-04-10 15:45:31]

Hi, PTAL

[hbos_chromium, 2017-04-10 15:50:43]

lgtm

[Guido Urdaneta, 2017-04-10 15:55:07]

The CQ bit was checked by guidou@chromium.org

[commit-bot: I haz the power, 2017-04-10 15:55:38]

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-04-10 17:26:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491839707418670, "parent_rev":
"63cf51ee5d04cd4d6bfb4299312214263a7c4a0a", "commit_rev":
"71ffa0d2e073d09d8098b0794f7f382655aabedc"}

[commit-bot: I haz the power, 2017-04-10 17:27:43]

Description was changed from

==========
Update selection of resolution policy for screen capture with getUserMedia.

The intent of the new algorithm is to allow resolution adjustment if the
resolution range allowed by constraints is a superset of the capabilities.
However, the actual capabilities are not currently known and, in practice,
resolution adjustment was allowed only when no constraints were specified.

This CL relaxes this rule to allow resolution adjustment by treating a max
constraint as if it were the actual capability. This should be updated so
that the actual native screen resolution is used as capability, but, in the
meantime, this fix is more compatible with existing applications that expect
the behavior of the old algorithm. Moreover, this patch does not affect spec
compliance since screen-resolution policies are not regulated by the spec as
long as the output resolution is within the constrained range.

BUG=709915
==========

to

==========
Update selection of resolution policy for screen capture with getUserMedia.

The intent of the new algorithm is to allow resolution adjustment if the
resolution range allowed by constraints is a superset of the capabilities.
However, the actual capabilities are not currently known and, in practice,
resolution adjustment was allowed only when no constraints were specified.

This CL relaxes this rule to allow resolution adjustment by treating a max
constraint as if it were the actual capability. This should be updated so
that the actual native screen resolution is used as capability, but, in the
meantime, this fix is more compatible with existing applications that expect
the behavior of the old algorithm. Moreover, this patch does not affect spec
compliance since screen-resolution policies are not regulated by the spec as
long as the output resolution is within the constrained range.

BUG=709915

Review-Url: https://codereview.chromium.org/2800343005
Cr-Commit-Position: refs/heads/master@{#463309}
Committed:
https://chromium.googlesource.com/chromium/src/+/71ffa0d2e073d09d8098b0794f7f...
==========

[commit-bot: I haz the power, 2017-04-10 17:27:45]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/71ffa0d2e073d09d8098b0794f7f...