Update Durable Storage API method signature

Update Durable Storage API method signature

Instead of returning a permission string, requestPersistent() was
recently changed to return a boolean representing the value of the default
box.

This is an experimental API only exposed behind a flag. The spec is at
https://storage.spec.whatwg.org/

BUG=482814
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200795

[dgrogan, 2015-08-19 00:53:31]

dgrogan@chromium.org changed reviewers:
+ jsbell@chromium.org

[dgrogan, 2015-08-19 00:53:31]

Josh, could you review this?

[jsbell, 2015-08-19 01:04:43]

lgtm with a string nit

can you update the CL description to mention that it's an experimental API, and
link to the spec?

https://codereview.chromium.org/1302643002/diff/1/LayoutTests/http/tests/stor...
File LayoutTests/http/tests/storage/durability-basics.html (right):

https://codereview.chromium.org/1302643002/diff/1/LayoutTests/http/tests/stor...
LayoutTests/http/tests/storage/durability-basics.html:18: assert_equals(typeof
result, "boolean", result + " should be a string");
" should be a string" -> " should be a boolean"

https://codereview.chromium.org/1302643002/diff/1/Source/modules/quota/OWNERS
File Source/modules/quota/OWNERS (right):

https://codereview.chromium.org/1302643002/diff/1/Source/modules/quota/OWNERS...
Source/modules/quota/OWNERS:1: jsbell@chromium.org
Usually requires the other owners to approve, doesn't it?

[dgrogan, 2015-08-19 01:12:33]

dgrogan@chromium.org changed reviewers:
+ kinuko@chromium.org

[dgrogan, 2015-08-19 01:12:34]

Kinuko, could you review this?

Josh reviewed the code. I added him to the OWNERS file. Is that ok?

https://codereview.chromium.org/1302643002/diff/1/LayoutTests/http/tests/stor...
File LayoutTests/http/tests/storage/durability-basics.html (right):

https://codereview.chromium.org/1302643002/diff/1/LayoutTests/http/tests/stor...
LayoutTests/http/tests/storage/durability-basics.html:18: assert_equals(typeof
result, "boolean", result + " should be a string");
On 2015/08/19 01:04:42, jsbell wrote:
> " should be a string" -> " should be a boolean"

Thanks, fixed.

https://codereview.chromium.org/1302643002/diff/1/Source/modules/quota/OWNERS
File Source/modules/quota/OWNERS (right):

https://codereview.chromium.org/1302643002/diff/1/Source/modules/quota/OWNERS...
Source/modules/quota/OWNERS:1: jsbell@chromium.org
On 2015/08/19 01:04:42, jsbell wrote:
> Usually requires the other owners to approve, doesn't it?

I think so. I will add Kinuko.

[kinuko, 2015-08-19 01:15:26]

Sure. lgtm

[dgrogan, 2015-08-19 02:25:46]

The CQ bit was checked by dgrogan@chromium.org

[dgrogan, 2015-08-19 02:25:46]

The patchset sent to the CQ was uploaded after l-g-t-m from jsbell@chromium.org
Link to the patchset: https://codereview.chromium.org/1302643002/#ps20001
(title: "Fix assert string")

[commit-bot: I haz the power, 2015-08-19 02:25:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1302643002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1302643002/20001

[commit-bot: I haz the power, 2015-08-19 03:07:14]

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

[commit-bot: I haz the power, 2015-08-19 03:07:15]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_arm64_d...)

[dgrogan, 2015-08-19 03:16:32]

The CQ bit was checked by dgrogan@chromium.org

[commit-bot: I haz the power, 2015-08-19 03:16:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1302643002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1302643002/20001

[commit-bot: I haz the power, 2015-08-19 04:38:33]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200795