Add requesting origin param to PermissionManager::GetPermissionStatusForFrame

Add requesting origin param to PermissionManager::GetPermissionStatusForFrame

GetPermissionStatusForFrame was currently added to facilitate permission
checks which require knowledge of the frame that is the source of the
permission request. Although we can determine the origin of that frame
directly through GetLastCommittedURL, we can't currently make the assumption
in Chrome that this origin matches the one passed down to various
callsites of GetPermissionStatus currently. crbug.com/698985 is
investigating that issue, however there are other reasons to have this
function, such as using it to incorporate Android permission checks and adding
permission delegation checks. Hence we pass in the requesting origin as well
for now and hopefully we can remove it again when crbug.com/698985 is resolved.

BUG=596786
Review-Url: https://codereview.chromium.org/2745453002
Cr-Commit-Position: refs/heads/master@{#456324}
Committed: https://chromium.googlesource.com/chromium/src/+/79f22a61dfa4d5a026bd3c75b8b04bbcdeae0c1d

[raymes, 2017-03-09 04:08:18]

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

[benwells, 2017-03-09 04:50:19]

lgtm

[raymes, 2017-03-12 23:34:14]

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

[commit-bot: I haz the power, 2017-03-12 23:34:21]

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-03-13 00:17:56]

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

[commit-bot: I haz the power, 2017-03-13 00:17:57]

Dry run: This issue passed the CQ dry run.

[raymes, 2017-03-13 04:28:31]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2017-03-13 04:28:39]

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-03-13 05:28:14]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489379311026220, "parent_rev":
"3a001533b29c0fe28f2354285b2c2ac8d57d8ce6", "commit_rev":
"79f22a61dfa4d5a026bd3c75b8b04bbcdeae0c1d"}

[commit-bot: I haz the power, 2017-03-13 05:29:29]

Description was changed from

==========
Add requesting origin param to PermissionManager::GetPermissionStatusForFrame

GetPermissionStatusForFrame was currently added to facilitate permission
checks which require knowledge of the frame that is the source of the
permission request. Although we can determine the origin of that frame
directly through GetLastCommittedURL, we can't currently make the assumption
in Chrome that this origin matches the one passed down to various
callsites of GetPermissionStatus currently. crbug.com/698985 is
investigating that issue, however there are other reasons to have this
function, such as using it to incorporate Android permission checks and adding
permission delegation checks. Hence we pass in the requesting origin as well
for now and hopefully we can remove it again when crbug.com/698985 is resolved.

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

to

==========
Add requesting origin param to PermissionManager::GetPermissionStatusForFrame

GetPermissionStatusForFrame was currently added to facilitate permission
checks which require knowledge of the frame that is the source of the
permission request. Although we can determine the origin of that frame
directly through GetLastCommittedURL, we can't currently make the assumption
in Chrome that this origin matches the one passed down to various
callsites of GetPermissionStatus currently. crbug.com/698985 is
investigating that issue, however there are other reasons to have this
function, such as using it to incorporate Android permission checks and adding
permission delegation checks. Hence we pass in the requesting origin as well
for now and hopefully we can remove it again when crbug.com/698985 is resolved.

BUG=596786

Review-Url: https://codereview.chromium.org/2745453002
Cr-Commit-Position: refs/heads/master@{#456324}
Committed:
https://chromium.googlesource.com/chromium/src/+/79f22a61dfa4d5a026bd3c75b8b0...
==========

[commit-bot: I haz the power, 2017-03-13 05:29:31]

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