Fix for default value of FetchEvent.cancellable

Fix for default value of FetchEvent.cancellable

new FetchEvent('foo').cancellable should be false
previously we were setting it to true in FetchEvent::create()
method.

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

[Paritosh Kumar, 2015-06-17 12:33:20]

paritosh.in@samsung.com changed reviewers:
+ falken@chromium.org, jsbell@chromium.org

[Paritosh Kumar, 2015-06-17 12:33:20]

Please have a look.

[Paritosh Kumar, 2015-06-17 12:34:15]

paritosh.in@samsung.com changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2015-06-17 12:51:31]

https://codereview.chromium.org/1187573003/diff/1/LayoutTests/http/tests/serv...
File LayoutTests/http/tests/serviceworker/resources/interfaces-worker.js
(right):

https://codereview.chromium.org/1187573003/diff/1/LayoutTests/http/tests/serv...
LayoutTests/http/tests/serviceworker/resources/interfaces-worker.js:85: false,
'Default FetchEvent.bubbles should be false');
why did this change? You only touched cancelable.

https://codereview.chromium.org/1187573003/diff/1/Source/modules/serviceworke...
File Source/modules/serviceworkers/FetchEvent.cpp (right):

https://codereview.chromium.org/1187573003/diff/1/Source/modules/serviceworke...
Source/modules/serviceworkers/FetchEvent.cpp:19:
PassRefPtrWillBeRawPtr<FetchEvent> FetchEvent::create(const AtomicString& type,
FetchEventInit& initializer)
FetchEventInit should be a const ref now

[jochen (gone - plz use gerrit), 2015-06-17 12:52:05]

also nit for cl description "default" instead of "defulat"

[Paritosh Kumar, 2015-06-17 13:15:32]

Thanks Jochen

Updated. Please have a look.

https://codereview.chromium.org/1187573003/diff/1/LayoutTests/http/tests/serv...
File LayoutTests/http/tests/serviceworker/resources/interfaces-worker.js
(right):

https://codereview.chromium.org/1187573003/diff/1/LayoutTests/http/tests/serv...
LayoutTests/http/tests/serviceworker/resources/interfaces-worker.js:85: false,
'Default FetchEvent.bubbles should be false');
On 2015/06/17 12:51:31, jochen wrote:
> why did this change? You only touched cancelable.

This was typo and corrected in a cl that get reverted. This should be false
only. If this is causing any confusion in this cl, then we can do in 
another cl. please suggest.

https://codereview.chromium.org/1187573003/diff/1/Source/modules/serviceworke...
File Source/modules/serviceworkers/FetchEvent.cpp (right):

https://codereview.chromium.org/1187573003/diff/1/Source/modules/serviceworke...
Source/modules/serviceworkers/FetchEvent.cpp:19:
PassRefPtrWillBeRawPtr<FetchEvent> FetchEvent::create(const AtomicString& type,
FetchEventInit& initializer)
On 2015/06/17 12:51:31, jochen wrote:
> FetchEventInit should be a const ref now

Yes. Thanks.

[jochen (gone - plz use gerrit), 2015-06-17 13:39:50]

ah, I see you changed the string only.

lgtm

[Paritosh Kumar, 2015-06-17 13:40:50]

The CQ bit was checked by paritosh.in@samsung.com

[commit-bot: I haz the power, 2015-06-17 13:41:25]

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

[commit-bot: I haz the power, 2015-06-17 14:43:41]

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