Pass the permission type to NotificationPermissionInfoBarDelegate

Pass the permission type to NotificationPermissionInfoBarDelegate

NotificationPermissionInfoBarDelegate is used for both the notifications
permission and for the push messaging permission. For correctness (mainly with
metrics right now), we need to pass the correct permission type through to the
PermissionInfobarDelegate.

BUG=644256
Committed: https://crrev.com/00d49400f8a7c4f189e132c00ed0118f606c4dc8
Cr-Commit-Position: refs/heads/master@{#430750}

[raymes, 2016-11-08 06:21:18]

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

[commit-bot: I haz the power, 2016-11-08 06:21:36]

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

[raymes, 2016-11-08 06:22:43]

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

[raymes, 2016-11-08 06:22:43]

Hi Peter - this is correcting the issue I raised in
https://codereview.chromium.org/2316673002

[commit-bot: I haz the power, 2016-11-08 06:52:14]

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

[commit-bot: I haz the power, 2016-11-08 06:52:14]

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

[raymes, 2016-11-08 06:58:41]

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

[commit-bot: I haz the power, 2016-11-08 06:58:51]

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, 2016-11-08 07:28:50]

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

[commit-bot: I haz the power, 2016-11-08 07:28:50]

Dry run: This issue passed the CQ dry run.

[Peter Beverloo, 2016-11-08 14:46:00]

lgtm, thanks!

Looks like the only non-UMA use of the value is ShouldShowPersistenceToggle(),
which isn't applicable to either permission types. (Nor would it make sense.)

[raymes, 2016-11-08 23:10:04]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2016-11-08 23:10:49]

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, 2016-11-08 23:17:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-08 23:38:15]

Description was changed from

==========
Pass the permission type to NotificationPermissionInfoBarDelegate

NotificationPermissionInfoBarDelegate is used for both the notifications
permission and for the push messaging permission. For correctness (mainly with
metrics right now), we need to pass the correct permission type through to the
PermissionInfobarDelegate.

BUG=644256
==========

to

==========
Pass the permission type to NotificationPermissionInfoBarDelegate

NotificationPermissionInfoBarDelegate is used for both the notifications
permission and for the push messaging permission. For correctness (mainly with
metrics right now), we need to pass the correct permission type through to the
PermissionInfobarDelegate.

BUG=644256

Committed: https://crrev.com/00d49400f8a7c4f189e132c00ed0118f606c4dc8
Cr-Commit-Position: refs/heads/master@{#430750}
==========

[commit-bot: I haz the power, 2016-11-08 23:38:16]

Patchset 2 (id:??) landed as
https://crrev.com/00d49400f8a7c4f189e132c00ed0118f606c4dc8
Cr-Commit-Position: refs/heads/master@{#430750}