[cleanup] Remove some WebPermissionType which are unused.

[cleanup] Remove some WebPermissionType which are unused.

Only 3 WebPermissionType types are used for implementing permission delegation.
(https://bugs.chromium.org/p/chromium/issues/detail?id=614608)
The rest can be removed.

BUG=none

Review-Url: https://codereview.chromium.org/2676973003
Cr-Commit-Position: refs/heads/master@{#448996}
Committed: https://chromium.googlesource.com/chromium/src/+/21c93b0db86d1d7aa881a7f08778c6f20a935406

[riju_, 2017-02-06 09:56:08]

The CQ bit was checked by rijubrata.bhaumik@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-06 09:56:22]

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-06 11:54:28]

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

[commit-bot: I haz the power, 2017-02-06 11:54:29]

Dry run: This issue passed the CQ dry run.

[riju_, 2017-02-06 12:12:24]

rijubrata.bhaumik@intel.com changed reviewers:
+ mlamouri@chromium.org

[riju_, 2017-02-06 12:12:25]

@mounir: PTAL.

[mlamouri (slow - plz ping), 2017-02-07 12:30:12]

Thanks for the CL. At the moment, WebPermissionType is mirroring the Mojo
permission types. Is there a specific reason why you would like to change this?

[riju_, 2017-02-07 12:48:11]

On 2017/02/07 12:30:12, mlamouri wrote:
> Thanks for the CL. At the moment, WebPermissionType is mirroring the Mojo
> permission types. Is there a specific reason why you would like to change
this?

Actually this came up when I was working on a different CL : Add Permission
guard to the generic sensor apis (https://codereview.chromium.org/2458453002/).
Just realised that the enum WebPermissionType is not used at all after the
mojo/Onion soup project.
So its basically cleanup of unused code.
If you think it look fine, I can try to remove the WebPermissionType.h file
altogether by making delegated permission use the generated permission mojo
files directly in the next CL.
Else, we can close this.

[mlamouri (slow - plz ping), 2017-02-08 12:25:45]

Thanks for the context. lgtm but I think it would be better to use the mojo type
and get ride of this header indeed.

[riju_, 2017-02-08 12:30:34]

The CQ bit was checked by rijubrata.bhaumik@intel.com

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

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 15:09:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486557034155930, "parent_rev":
"2176e15afc8a82813c303e3001a02aad930246a2", "commit_rev":
"21c93b0db86d1d7aa881a7f08778c6f20a935406"}

[commit-bot: I haz the power, 2017-02-08 15:09:39]

Description was changed from

==========
[cleanup] Remove some WebPermissionType which are unused.

Only 3 WebPermissionType types are used for implementing permission delegation.
(https://bugs.chromium.org/p/chromium/issues/detail?id=614608)
The rest can be removed.

BUG=none
==========

to

==========
[cleanup] Remove some WebPermissionType which are unused.

Only 3 WebPermissionType types are used for implementing permission delegation.
(https://bugs.chromium.org/p/chromium/issues/detail?id=614608)
The rest can be removed.

BUG=none

Review-Url: https://codereview.chromium.org/2676973003
Cr-Commit-Position: refs/heads/master@{#448996}
Committed:
https://chromium.googlesource.com/chromium/src/+/21c93b0db86d1d7aa881a7f08778...
==========

[commit-bot: I haz the power, 2017-02-08 15:09:40]

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