Remove the MediaPermissionStatus helper class

Remove the MediaPermissionStatus helper class

Originally MediaPermissionStatus was added to facilitate moving some of
the permissions logic out of MediaStreamDevicesController. It turns out
that it is simpler to move the permission request logic out instead, as
is done in the followup CL: https://codereview.chromium.org/2811913002

This CL removes that helper class and moves the functionality back into
GetContentSetting.

BUG=596786
Review-Url: https://codereview.chromium.org/2810743004
Cr-Commit-Position: refs/heads/master@{#465111}
Committed: https://chromium.googlesource.com/chromium/src/+/799cbeb7126756f38d29d4bdf88a2efe417ad862

[raymes, 2017-04-10 23:20:52]

Description was changed from

==========
Remove the MediaPermissionStatus helper class

Originally MediaPermissionStatus was added to facilitate moving some of
the permissions logic out of MediaStreamDevicesController. It turns out
that it is simpler to move the permission request logic out instead, as
is done in the followup CL:

This CL removes that helper class and moves the functionality back into
GetContentSetting.

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

to

==========
Remove the MediaPermissionStatus helper class

Originally MediaPermissionStatus was added to facilitate moving some of
the permissions logic out of MediaStreamDevicesController. It turns out
that it is simpler to move the permission request logic out instead, as
is done in the followup CL: https://codereview.chromium.org/2811913002

This CL removes that helper class and moves the functionality back into
GetContentSetting.

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

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

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

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

Sorry for the churn on this, but if you look at the followup CL I think the
direction will become clearer.

[raymes, 2017-04-11 03:30:22]

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

[commit-bot: I haz the power, 2017-04-11 03:30:37]

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 04:32:29]

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

[commit-bot: I haz the power, 2017-04-11 04:32:30]

Dry run: This issue passed the CQ dry run.

[raymes, 2017-04-13 01:53:16]

Ping :)

[Timothy Loh, 2017-04-18 01:27:36]

On 2017/04/13 01:53:16, raymes wrote:
> Ping :)

lgtm

[raymes, 2017-04-18 01:51:03]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2017-04-18 01:51:41]

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-18 02:38:03]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492480263407370,
"parent_rev": "8dbd95ed5e1531af5df3d342edaf0e8b31ef8144", "commit_rev":
"799cbeb7126756f38d29d4bdf88a2efe417ad862"}

[commit-bot: I haz the power, 2017-04-18 02:38:11]

Description was changed from

==========
Remove the MediaPermissionStatus helper class

Originally MediaPermissionStatus was added to facilitate moving some of
the permissions logic out of MediaStreamDevicesController. It turns out
that it is simpler to move the permission request logic out instead, as
is done in the followup CL: https://codereview.chromium.org/2811913002

This CL removes that helper class and moves the functionality back into
GetContentSetting.

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

to

==========
Remove the MediaPermissionStatus helper class

Originally MediaPermissionStatus was added to facilitate moving some of
the permissions logic out of MediaStreamDevicesController. It turns out
that it is simpler to move the permission request logic out instead, as
is done in the followup CL: https://codereview.chromium.org/2811913002

This CL removes that helper class and moves the functionality back into
GetContentSetting.

BUG=596786

Review-Url: https://codereview.chromium.org/2810743004
Cr-Commit-Position: refs/heads/master@{#465111}
Committed:
https://chromium.googlesource.com/chromium/src/+/799cbeb7126756f38d29d4bdf88a...
==========

[commit-bot: I haz the power, 2017-04-18 02:38:12]

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