Use spec-compliant algorithm to select video devices in getUserMedia.

Use spec-compliant algorithm to select video devices in getUserMedia.

BUG=657733
Review-Url: https://codereview.chromium.org/2669243004
Cr-Commit-Position: refs/heads/master@{#448981}
Committed: https://chromium.googlesource.com/chromium/src/+/9ea409c2a23810063bb280304e18b48dc6b0fe7e

[Guido Urdaneta, 2017-02-02 17:35:40]

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

[commit-bot: I haz the power, 2017-02-02 17:36:03]

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

[Guido Urdaneta, 2017-02-02 19:51:59]

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

[commit-bot: I haz the power, 2017-02-02 19:52:19]

Dry run: 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-02-02 22:51:26]

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

[commit-bot: I haz the power, 2017-02-02 22:51:27]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Guido Urdaneta, 2017-02-03 16:09:35]

Patchset #3 (id:40001) has been deleted

[Guido Urdaneta, 2017-02-03 16:09:42]

Patchset #2 (id:20001) has been deleted

[Guido Urdaneta, 2017-02-03 16:09:49]

Patchset #1 (id:1) has been deleted

[Guido Urdaneta, 2017-02-03 16:42:32]

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

[commit-bot: I haz the power, 2017-02-03 16:43:01]

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

[Guido Urdaneta, 2017-02-03 16:44:43]

guidou@chromium.org changed reviewers:
+ mek@chromium.org, tommi@chromium.org

[Guido Urdaneta, 2017-02-03 16:44:44]

Hi, PTAL

mek@chromium.org: Please review changes in extensions/

tommi@chromium.org: Please review changes in content/

[Marijn Kruisselbrink, 2017-02-03 17:39:15]

extensions lgtm

[commit-bot: I haz the power, 2017-02-03 17:54:24]

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

[commit-bot: I haz the power, 2017-02-03 17:54:25]

Dry run: This issue passed the CQ dry run.

[tommi (sloooow) - chröme, 2017-02-03 18:03:06]

lgtm

https://codereview.chromium.org/2669243004/diff/60001/content/renderer/media/...
File content/renderer/media/user_media_client_impl.cc (right):

https://codereview.chromium.org/2669243004/diff/60001/content/renderer/media/...
content/renderer/media/user_media_client_impl.cc:242:
worker_task_runner_(worker_task_runner),
can worker_task_runner_ be const?

[Guido Urdaneta, 2017-02-03 21:33:04]

https://codereview.chromium.org/2669243004/diff/60001/content/renderer/media/...
File content/renderer/media/user_media_client_impl.cc (right):

https://codereview.chromium.org/2669243004/diff/60001/content/renderer/media/...
content/renderer/media/user_media_client_impl.cc:242:
worker_task_runner_(worker_task_runner),
On 2017/02/03 18:03:06, tommi (krómíum) wrote:
> can worker_task_runner_ be const?

Done.

[Guido Urdaneta, 2017-02-03 21:34:21]

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

[Guido Urdaneta, 2017-02-03 21:34:22]

kinuko@chromium.org: Can you take a look or rs
content/renderer/render_frame_impl.cc?

[kinuko, 2017-02-04 00:02:48]

lgtm

[Guido Urdaneta, 2017-02-08 12:46:53]

The CQ bit was checked by guidou@chromium.org

[Guido Urdaneta, 2017-02-08 12:46:53]

The patchset sent to the CQ was uploaded after l-g-t-m from mek@chromium.org,
tommi@chromium.org
Link to the patchset: https://codereview.chromium.org/2669243004/#ps80001
(title: "address tommi's comment")

[commit-bot: I haz the power, 2017-02-08 12:47:12]

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-02-08 13:54:16]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1486558013187140,
"parent_rev": "81022874e21cac29a6d57ac820ef26f479700fc6", "commit_rev":
"9ea409c2a23810063bb280304e18b48dc6b0fe7e"}

[commit-bot: I haz the power, 2017-02-08 13:54:51]

Description was changed from

==========
Use spec-compliant algorithm to select video devices in getUserMedia.

BUG=657733
==========

to

==========
Use spec-compliant algorithm to select video devices in getUserMedia.

BUG=657733

Review-Url: https://codereview.chromium.org/2669243004
Cr-Commit-Position: refs/heads/master@{#448981}
Committed:
https://chromium.googlesource.com/chromium/src/+/9ea409c2a23810063bb280304e18...
==========

[commit-bot: I haz the power, 2017-02-08 13:54:54]

Committed patchset #2 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/9ea409c2a23810063bb280304e18...

[perezju, 2017-02-10 12:49:06]

A revert of this CL (patchset #2 id:80001) has been created in
https://codereview.chromium.org/2684233005/ by perezju@chromium.org.

The reason for reverting is: Broke
org.chromium.android_webview.test.MediaAccessPermissionRequestTest
https://crbug.com/690626.