Image Capture: Reject applyConstraints() if passed an unsupported constraint

Image Capture: Reject applyConstraints() if passed an unsupported constraint

This CL adds a check to reject applyConstraints() if it is passed an
unsupported constraint. LayoutTests gets an updated, albeit a bit a
cumbersome one because they hit an unrelated bug (https://crbug.com/711524).

BUG=711694
Review-Url: https://codereview.chromium.org/2819653003
Cr-Commit-Position: refs/heads/master@{#464820}
(cherry picked from commit 635372d1d6a0318a44dbc1be736d8060a126d68a)


NOTRY=true 
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2819233002
Cr-Commit-Position: refs/branch-heads/3071@{#13}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
Committed: https://chromium.googlesource.com/chromium/src/+/dfb6460559869ea71010b5c5df456e63858d401b

[mcasas, 2017-04-17 15:00:04]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2017-04-17 15:00: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-04-17 15:00:12]

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

[commit-bot: I haz the power, 2017-04-17 15:00:13]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[mcasas, 2017-04-17 15:01:05]

mcasas@chromium.org changed reviewers:
+ reillyg@chromium.org

[mcasas, 2017-04-17 15:01:05]

reillyg@ RS plz  (this CL is a merge-to-branch)

[haraken, 2017-04-17 15:01:59]

rs LGTM

[mcasas, 2017-04-17 15:25:37]

The CQ bit was checked by mcasas@chromium.org

[mcasas, 2017-04-17 15:25:45]

On 2017/04/17 15:01:59, haraken wrote:
> rs LGTM

thanks haraken@ !

[commit-bot: I haz the power, 2017-04-17 15:25:45]

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-17 15:25:46]

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

[commit-bot: I haz the power, 2017-04-17 15:25:47]

CLs for remote refs other than refs/heads/master must contain NOTRY=true and
NOPRESUBMIT=true in order for the CQ to process them

[mcasas, 2017-04-17 15:26:16]

Description was changed from

==========
Image Capture: Reject applyConstraints() if passed an unsupported constraint

This CL adds a check to reject applyConstraints() if it is passed an
unsupported constraint. LayoutTests gets an updated, albeit a bit a
cumbersome one because they hit an unrelated bug (https://crbug.com/711524).

BUG=711694

Review-Url: https://codereview.chromium.org/2819653003
Cr-Commit-Position: refs/heads/master@{#464820}
(cherry picked from commit 635372d1d6a0318a44dbc1be736d8060a126d68a)
==========

to

==========
Image Capture: Reject applyConstraints() if passed an unsupported constraint

This CL adds a check to reject applyConstraints() if it is passed an
unsupported constraint. LayoutTests gets an updated, albeit a bit a
cumbersome one because they hit an unrelated bug (https://crbug.com/711524).

BUG=711694

Review-Url: https://codereview.chromium.org/2819653003
Cr-Commit-Position: refs/heads/master@{#464820}
(cherry picked from commit 635372d1d6a0318a44dbc1be736d8060a126d68a)


NOTRY=true 
NOPRESUBMIT=true
==========

[mcasas, 2017-04-17 15:26:21]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2017-04-17 15:26: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-17 15:27:09]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492442781609880, "parent_rev":
"db95542618183726d9b0193c1178258d5f95695a", "commit_rev":
"dfb6460559869ea71010b5c5df456e63858d401b"}

[commit-bot: I haz the power, 2017-04-17 15:27:19]

Description was changed from

==========
Image Capture: Reject applyConstraints() if passed an unsupported constraint

This CL adds a check to reject applyConstraints() if it is passed an
unsupported constraint. LayoutTests gets an updated, albeit a bit a
cumbersome one because they hit an unrelated bug (https://crbug.com/711524).

BUG=711694

Review-Url: https://codereview.chromium.org/2819653003
Cr-Commit-Position: refs/heads/master@{#464820}
(cherry picked from commit 635372d1d6a0318a44dbc1be736d8060a126d68a)


NOTRY=true 
NOPRESUBMIT=true
==========

to

==========
Image Capture: Reject applyConstraints() if passed an unsupported constraint

This CL adds a check to reject applyConstraints() if it is passed an
unsupported constraint. LayoutTests gets an updated, albeit a bit a
cumbersome one because they hit an unrelated bug (https://crbug.com/711524).

BUG=711694

Review-Url: https://codereview.chromium.org/2819653003
Cr-Commit-Position: refs/heads/master@{#464820}
(cherry picked from commit 635372d1d6a0318a44dbc1be736d8060a126d68a)


NOTRY=true 
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2819233002
Cr-Commit-Position: refs/branch-heads/3071@{#13}
Cr-Branched-From:
a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
Committed:
https://chromium.googlesource.com/chromium/src/+/dfb6460559869ea71010b5c5df45...
==========

[commit-bot: I haz the power, 2017-04-17 15:27:20]

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

[Reilly Grant (use Gerrit), 2017-04-17 15:43:59]

lgtm but if you have merge approval you can generally TBR