Only enable kUsePermissionManagerForMediaRequests when the PermissionRequestManager is used

Only enable kUsePermissionManagerForMediaRequests when the PermissionRequestManager is used

This adds a check which ensures that the PermissionManager is used to
request permission for Media requests only when the
PermissionRequestManager is enabled. If it's not enabled, the grouped
request won't work correctly.

BUG=596786
Review-Url: https://codereview.chromium.org/2941943002
Cr-Commit-Position: refs/heads/master@{#480017}
Committed: https://chromium.googlesource.com/chromium/src/+/9ea317b97a564286aadf81bfe3d2bcb1d0e642cc

[raymes, 2017-06-15 04:57:51]

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

[raymes, 2017-06-15 04:57:58]

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

[commit-bot: I haz the power, 2017-06-15 04: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-06-15 05:25:54]

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

[commit-bot: I haz the power, 2017-06-15 05:25:55]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[raymes, 2017-06-15 05:57:24]

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

[commit-bot: I haz the power, 2017-06-15 05:57:39]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Timothy Loh, 2017-06-15 06:57:55]

On 2017/06/15 04:57:52, raymes wrote:

lgtm

[commit-bot: I haz the power, 2017-06-15 07:01:23]

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

[commit-bot: I haz the power, 2017-06-15 07:01:24]

Dry run: This issue passed the CQ dry run.

[raymes, 2017-06-16 10:02:19]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2017-06-16 10:02:26]

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-06-16 11:10:17]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1497607339335050,
"parent_rev": "22fc506b67c1bc65dead0bd636e84d1a616b9986", "commit_rev":
"9ea317b97a564286aadf81bfe3d2bcb1d0e642cc"}

[commit-bot: I haz the power, 2017-06-16 11:10:28]

Description was changed from

==========
Only enable kUsePermissionManagerForMediaRequests when the
PermissionRequestManager is used

This adds a check which ensures that the PermissionManager is used to
request permission for Media requests only when the
PermissionRequestManager is enabled. If it's not enabled, the grouped
request won't work correctly.

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

to

==========
Only enable kUsePermissionManagerForMediaRequests when the
PermissionRequestManager is used

This adds a check which ensures that the PermissionManager is used to
request permission for Media requests only when the
PermissionRequestManager is enabled. If it's not enabled, the grouped
request won't work correctly.

BUG=596786

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

[commit-bot: I haz the power, 2017-06-16 11:10:28]

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