[ServiceWorker] Add was_fallback_required flag to ResourceResponseInfo. [2/2 chromium]

[ServiceWorker] Add was_fallback_required flag to ResourceResponseInfo. [2/2 chromium]

This flag is set when the request whose mode is CORS or CORS-with-forced-preflight is sent to a ServiceWorker but FetchEvent.respondWith is not called.
It is because the CORS preflight logic is implemented in the renderer.
So we can't simply fallback to the network in the browser process.

[1/2] blink: https://codereview.chromium.org/602073002/
[2/2] chromium: https://codereview.chromium.org/603913002/

This change depends on https://codereview.chromium.org/588153002/.

BUG=416371, 408507
Committed: https://crrev.com/00320571a4696025c2216a4f1fe193f13c5f2355
Cr-Commit-Position: refs/heads/master@{#297610}

[horo, 2014-09-25 10:54:31]

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

[horo, 2014-09-25 10:54:31]

yhirano@
Could you please review?

[horo, 2014-09-26 01:28:46]

Patchset #2 (id:20001) has been deleted

[yhirano, 2014-09-26 04:27:25]

"which mode is CORS" in the description should be "whose mode is CORS".

https://codereview.chromium.org/603913002/diff/60001/content/public/common/re...
File content/public/common/resource_response_info.h (right):

https://codereview.chromium.org/603913002/diff/60001/content/public/common/re...
content/public/common/resource_response_info.h:104: // True when the request
which mode is |CORS| or |CORS-with-forced-preflight|
s/which/whose/

[horo, 2014-09-26 05:07:58]

done

https://codereview.chromium.org/603913002/diff/60001/content/public/common/re...
File content/public/common/resource_response_info.h (right):

https://codereview.chromium.org/603913002/diff/60001/content/public/common/re...
content/public/common/resource_response_info.h:104: // True when the request
which mode is |CORS| or |CORS-with-forced-preflight|
On 2014/09/26 04:27:25, yhirano wrote:
> s/which/whose/

Done.

[yhirano, 2014-09-26 05:50:06]

lgtm

[horo, 2014-09-26 07:33:55]

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

[horo, 2014-09-26 07:33:55]

nasko@
Could you please review content/common/resource_messages.h?

[nasko, 2014-09-26 13:57:12]

IPC LGTM

[horo, 2014-09-30 13:47:59]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-09-30 13:49:09]

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

[commit-bot: I haz the power, 2014-09-30 13:57:07]

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

[commit-bot: I haz the power, 2014-09-30 13:57:08]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[horo, 2014-09-30 16:03:04]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-09-30 16:03:55]

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

[commit-bot: I haz the power, 2014-09-30 16:12:27]

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

[commit-bot: I haz the power, 2014-09-30 16:12:28]

Try jobs failed on following builders:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/62525)
  android_chromium_gn_compile_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[horo, 2014-09-30 22:22:33]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-09-30 22:23:13]

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

[commit-bot: I haz the power, 2014-09-30 22:33:35]

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

[commit-bot: I haz the power, 2014-09-30 22:33:37]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[horo, 2014-10-01 00:18:16]

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

[horo, 2014-10-01 00:18:17]

jochen@
Could you please review this?

[jochen (gone - plz use gerrit), 2014-10-01 07:47:01]

lgtm

[horo, 2014-10-01 07:49:40]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-10-01 07:50:28]

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

[commit-bot: I haz the power, 2014-10-01 07:57:04]

Committed patchset #4 (id:80001) as b6f00aadcafeb26283667a39145e01bceae9d834

[commit-bot: I haz the power, 2014-10-01 07:58:07]

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