[ServiceWorker] Don't pass OPTIONS request to the ServiceWorker

[ServiceWorker] Don't pass OPTIONS request to the ServiceWorker

This is work around to avoid hijacking CORS preflight.
In current implementation, CORS preflight request is also sent to the ServiceWorker.
This is a bug in the implementation of HTTP fetch in DocumentThreadableLoader.
But it takes time to fix this.
So I want to introduce this work around because this is a security bug.

BUG=409454
Committed: https://crrev.com/2c8acfe2e520a5a44e3d158441dbc2bd2057a899
Cr-Commit-Position: refs/heads/master@{#293697}

[horo, 2014-09-06 04:58:07]

Patchset #1 (id:1) has been deleted

[horo, 2014-09-06 06:22:05]

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

[horo, 2014-09-06 06:22:06]

yhirano@
Could you please review this?

[yhirano, 2014-09-08 03:42:32]

Can we have a test?

[horo, 2014-09-08 06:21:10]

Patchset #3 (id:60001) has been deleted

[horo, 2014-09-08 06:21:54]

On 2014/09/08 03:42:32, yhirano wrote:
> Can we have a test?

Added test

[yhirano, 2014-09-08 07:58:01]

lgtm

https://codereview.chromium.org/548773002/diff/80001/content/browser/service_...
File content/browser/service_worker/service_worker_request_handler_unittest.cc
(right):

https://codereview.chromium.org/548773002/diff/80001/content/browser/service_...
content/browser/service_worker/service_worker_request_handler_unittest.cc:11:
#include "content/browser/service_worker/service_worker_request_handler.h"
Can you move this include to the top?

[horo, 2014-09-08 08:13:25]

https://codereview.chromium.org/548773002/diff/80001/content/browser/service_...
File content/browser/service_worker/service_worker_request_handler_unittest.cc
(right):

https://codereview.chromium.org/548773002/diff/80001/content/browser/service_...
content/browser/service_worker/service_worker_request_handler_unittest.cc:11:
#include "content/browser/service_worker/service_worker_request_handler.h"
On 2014/09/08 07:58:01, yhirano wrote:
> Can you move this include to the top?

Done.

[horo, 2014-09-08 08:13:59]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-09-08 08:14:11]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/horo@chromium.org/548773002/100001

[commit-bot: I haz the power, 2014-09-08 09:46:53]

Committed patchset #4 (id:100001) as 5463cf6821152b9f8963fad8f62d953b78b13e98

[commit-bot: I haz the power, 2014-09-10 03:45:26]

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