Change MediaStreamDevicesController::RequestPermissions to use the requesting WebContents

Change MediaStreamDevicesController::RequestPermissions to use the requesting WebContents

This changes MediaStreamDevicesController::RequestPermissions to use the
WebContents associated with the RenderFrameHost specified in the
request. This is important because we will be switching
RequestPermissions to call into PermissionManager::GetPermissionStatus
and PermissionManager::RequestPermissions which will take a
RenderFrameHost and not a WebContents. Some tests are fixed to work
correctly.

BUG=596786
Review-Url: https://codereview.chromium.org/2746853002
Cr-Commit-Position: refs/heads/master@{#463467}
Committed: https://chromium.googlesource.com/chromium/src/+/3bc708ecbaf8b268843a706e50de59a7bc7905f2

[raymes, 2017-03-14 04:27:46]

raymes@chromium.org changed reviewers:
+ timloh@chromium.org

[Timothy Loh, 2017-03-16 04:59:37]

On 2017/03/14 04:27:46, raymes wrote:

lgtm

[raymes, 2017-04-10 00:34:09]

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

[commit-bot: I haz the power, 2017-04-10 00:34:17]

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

[raymes, 2017-04-10 00:35:35]

raymes@chromium.org changed reviewers:
+ pastarmovj@chromium.org, sergeyu@chromium.org

[raymes, 2017-04-10 00:35:36]

+OWNERS
sergeyu for
chrome/browser/media/webrtc/permission_bubble_media_access_handler.cc
pastarmovj for chrome/browser/policy/policy_browsertest.cc

[commit-bot: I haz the power, 2017-04-10 00:46:53]

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

[commit-bot: I haz the power, 2017-04-10 00:46:54]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[raymes, 2017-04-10 00:57:15]

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

[commit-bot: I haz the power, 2017-04-10 00:57:30]

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-04-10 01:09:18]

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

[commit-bot: I haz the power, 2017-04-10 01:09:19]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[pastarmovj, 2017-04-10 08:13:40]

policy lgtm

[Sergey Ulanov, 2017-04-10 18:48:09]

lgtm

[raymes, 2017-04-10 22:23:07]

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

[commit-bot: I haz the power, 2017-04-10 22:23:43]

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-04-10 22:41:01]

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

[commit-bot: I haz the power, 2017-04-10 22:41:02]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[raymes, 2017-04-10 22:57:32]

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

[commit-bot: I haz the power, 2017-04-10 22:58:12]

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-04-11 00:04:52]

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

[commit-bot: I haz the power, 2017-04-11 00:04:53]

Dry run: This issue passed the CQ dry run.

[raymes, 2017-04-11 00:23:23]

The CQ bit was checked by raymes@chromium.org

[raymes, 2017-04-11 00:23:24]

The patchset sent to the CQ was uploaded after l-g-t-m from timloh@chromium.org,
sergeyu@chromium.org, pastarmovj@chromium.org
Link to the patchset: https://codereview.chromium.org/2746853002/#ps90001
(title: "Change MediaStreamDevicesController::RequestPermissions to use the
requesting WebContents")

[commit-bot: I haz the power, 2017-04-11 00:24:19]

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-11 00:38:28]

CQ is committing da patch.

Bot data: {"patchset_id": 90001, "attempt_start_ts": 1491870203966930,
"parent_rev": "65d6cd8be033362397ccd6d444cefc8a4be2fd1c", "commit_rev":
"3bc708ecbaf8b268843a706e50de59a7bc7905f2"}

[commit-bot: I haz the power, 2017-04-11 00:39:24]

Description was changed from

==========
Change MediaStreamDevicesController::RequestPermissions to use the requesting
WebContents

This changes MediaStreamDevicesController::RequestPermissions to use the
WebContents associated with the RenderFrameHost specified in the
request. This is important because we will be switching
RequestPermissions to call into PermissionManager::GetPermissionStatus
and PermissionManager::RequestPermissions which will take a
RenderFrameHost and not a WebContents. Some tests are fixed to work
correctly.

BUG=596786
==========

to

==========
Change MediaStreamDevicesController::RequestPermissions to use the requesting
WebContents

This changes MediaStreamDevicesController::RequestPermissions to use the
WebContents associated with the RenderFrameHost specified in the
request. This is important because we will be switching
RequestPermissions to call into PermissionManager::GetPermissionStatus
and PermissionManager::RequestPermissions which will take a
RenderFrameHost and not a WebContents. Some tests are fixed to work
correctly.

BUG=596786

Review-Url: https://codereview.chromium.org/2746853002
Cr-Commit-Position: refs/heads/master@{#463467}
Committed:
https://chromium.googlesource.com/chromium/src/+/3bc708ecbaf8b268843a706e50de...
==========

[commit-bot: I haz the power, 2017-04-11 00:39:25]

Committed patchset #6 (id:90001) as
https://chromium.googlesource.com/chromium/src/+/3bc708ecbaf8b268843a706e50de...