[ServiceWorker] support OPTIONS request for ServiceWorker [3/3 blink]

[ServiceWorker] support OPTIONS request for ServiceWorker [3/3 blink]

Currently we can't handle OPTIONS request in the ServiceWorker.

This check is done in content/browser/service_worker/service_worker_request_handler.cc.
But we should support OPTIONS request except for CORS preflight request.

[1/3] blink: https://codereview.chromium.org/738033002/
Set SkipServiceWorker flag for CORS preflight request.
[2/3] chromium: https://codereview.chromium.org/716083003/
Remove IsMethodSupportedForServiceWorker check in service_worker_request_handler.cc
[3/3] blink: https://codereview.chromium.org/743653002/
Cleanup and add update LayoutTest.

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

[horo, 2014-11-19 12:43:14]

horo@chromium.org changed reviewers:
+ yhirano@chromium.org

[horo, 2014-11-19 12:43:15]

yhirano@
Could you please review this?

[horo, 2014-11-21 02:05:47]

Patchset #3 (id:40001) has been deleted

[yhirano, 2014-11-21 03:44:51]

lgtm

https://codereview.chromium.org/743653002/diff/60001/LayoutTests/http/tests/s...
File
LayoutTests/http/tests/serviceworker/resources/fetch-request-xhr-iframe.html
(right):

https://codereview.chromium.org/743653002/diff/60001/LayoutTests/http/tests/s...
LayoutTests/http/tests/serviceworker/resources/fetch-request-xhr-iframe.html:84:
.then(function(response){
please add a space after between ')' and '{'

[yhirano, 2014-11-21 03:51:45]

https://codereview.chromium.org/743653002/diff/60001/LayoutTests/http/tests/s...
File
LayoutTests/http/tests/serviceworker/resources/fetch-request-xhr-iframe.html
(right):

https://codereview.chromium.org/743653002/diff/60001/LayoutTests/http/tests/s...
LayoutTests/http/tests/serviceworker/resources/fetch-request-xhr-iframe.html:84:
.then(function(response){
On 2014/11/21 03:44:51, yhirano wrote:
> please add a space after between ')' and '{'

Sorry, I meant
 please add a space between ')' and '{'

[horo, 2014-11-21 04:22:49]

Thank you!

https://codereview.chromium.org/743653002/diff/60001/LayoutTests/http/tests/s...
File
LayoutTests/http/tests/serviceworker/resources/fetch-request-xhr-iframe.html
(right):

https://codereview.chromium.org/743653002/diff/60001/LayoutTests/http/tests/s...
LayoutTests/http/tests/serviceworker/resources/fetch-request-xhr-iframe.html:84:
.then(function(response){
On 2014/11/21 03:51:45, yhirano wrote:
> On 2014/11/21 03:44:51, yhirano wrote:
> > please add a space after between ')' and '{'
> 
> Sorry, I meant
>  please add a space between ')' and '{'

Done.

[horo, 2014-11-21 04:24:12]

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

[horo, 2014-11-21 04:24:12]

mkwst@
Could you please review this?

[Mike West, 2014-11-21 08:25:18]

LGTM.

[horo, 2014-11-22 02:55:32]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-11-22 02:56:17]

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

[commit-bot: I haz the power, 2014-11-22 04:08:02]

Committed patchset #4 (id:80001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=185836