Rename hasPermission to permissionState.

Rename hasPermission to permissionState.

As per https://github.com/w3c/push-api/issues/136
BUG=479246
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194410

[Miguel Garcia, 2015-04-22 09:55:42]

Patchset #1 (id:1) has been deleted

[Miguel Garcia, 2015-04-22 09:55:52]

Patchset #1 (id:20001) has been deleted

[Miguel Garcia, 2015-04-22 17:15:00]

miguelg@chromium.org changed reviewers:
+ mvanouwerkerk@chromium.org

[Miguel Garcia, 2015-04-22 17:15:43]

This follows the spec change agreed on 
https://github.com/w3c/push-api/issues/136

[Michael van Ouwerkerk, 2015-04-22 17:45:40]

This change lgtm but if you submit it now, the PushMessagingBrowserTest will
start failing because it still calls hasPermission. You need to either
temporarily support both (playing nice) or temporarily disable the relevant
tests and then update them and re-enable after the blink roll.

See:
chrome/test/data/push_messaging/push_test.js
chrome/browser/push_messaging/push_messaging_browsertest.cc

[Miguel Garcia, 2015-04-24 11:25:36]

miguelg@chromium.org changed reviewers:
+ mkwst@chromium.org

[Miguel Garcia, 2015-04-24 11:25:54]

Adding mkwst for Source/platform/RuntimeEnabledFeatures.in OWNERS

[Mike West, 2015-04-24 12:32:22]

LGTM.

[Miguel Garcia, 2015-04-24 12:34:11]

The CQ bit was checked by miguelg@chromium.org

[Miguel Garcia, 2015-04-24 12:34:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
mvanouwerkerk@chromium.org
Link to the patchset: https://codereview.chromium.org/1083193006/#ps100001
(title: "Revert change in RuntimeEnabledFeatures.in")

[commit-bot: I haz the power, 2015-04-24 12:34:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1083193006/100001

[commit-bot: I haz the power, 2015-04-24 15:04:33]

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

[commit-bot: I haz the power, 2015-04-24 15:04:34]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/60697)

[Miguel Garcia, 2015-04-24 15:24:32]

The CQ bit was checked by miguelg@chromium.org

[Miguel Garcia, 2015-04-24 15:24:32]

The patchset sent to the CQ was uploaded after l-g-t-m from mkwst@chromium.org,
mvanouwerkerk@chromium.org
Link to the patchset: https://codereview.chromium.org/1083193006/#ps120001
(title: "re-add hasPermission to the expectations file")

[commit-bot: I haz the power, 2015-04-24 15:25:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1083193006/120001

[commit-bot: I haz the power, 2015-04-24 16:35:29]

Committed patchset #5 (id:120001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194410