ServiceWorker: Reland of bypassing SW when no fetch handler existed

ServiceWorker: Reland of bypassing SW when no fetch handler existed

When the script doesn't have fetch handler, in-scope network requests
don't need to go to the service worker. This patch is to bypass it.
This patch is a CL for re-landing which was reverted at
http://crrev.com/2080793003.

This CL should be landed after http://crrev.com/2108573002 is committed.

BUG=605844, 621788
TEST=./out/Debug/content_unittests --gtest_filter="ServiceWorkerControlleeRequestHandlerTest.FallbackWithNoFetchHandler*"

Committed: https://crrev.com/cb53dfa21239d8be2ed3bc53c83d8c001603cee6
Cr-Commit-Position: refs/heads/master@{#404348}

[shimazu, 2016-06-28 08:33:53]

Description was changed from

==========
ServiceWorker: reland of bypassing SW when no fetch handler existed

When the script doesn't have fetch handler, in-scope network requests
don't need to go to the service worker. This patch is to bypass it.
This patch is a CL for re-landing which was reverted at
http://crrev.com/2080793003.

This CL should be landed after http://crrev.com/2108573002 is committed.

BUG=605844,621788
TEST=./out/Debug/content_unittests
--gtest_filter="ServiceWorkerControlleeRequestHandlerTest.FallbackWithNoFetchHandler*"
==========

to

==========
ServiceWorker: reland of bypassing SW when no fetch handler existed

When the script doesn't have fetch handler, in-scope network requests
don't need to go to the service worker. This patch is to bypass it.
This patch is a CL for re-landing which was reverted at
http://crrev.com/2080793003.

This CL should be landed after http://crrev.com/2108573002 is committed.

BUG=605844,621788
TEST=./out/Debug/content_unittests
--gtest_filter="ServiceWorkerControlleeRequestHandlerTest.FallbackWithNoFetchHandler*"
==========

[shimazu, 2016-06-28 08:34:04]

Description was changed from

==========
ServiceWorker: reland of bypassing SW when no fetch handler existed

When the script doesn't have fetch handler, in-scope network requests
don't need to go to the service worker. This patch is to bypass it.
This patch is a CL for re-landing which was reverted at
http://crrev.com/2080793003.

This CL should be landed after http://crrev.com/2108573002 is committed.

BUG=605844,621788
TEST=./out/Debug/content_unittests
--gtest_filter="ServiceWorkerControlleeRequestHandlerTest.FallbackWithNoFetchHandler*"
==========

to

==========
ServiceWorker: Reland of bypassing SW when no fetch handler existed

When the script doesn't have fetch handler, in-scope network requests
don't need to go to the service worker. This patch is to bypass it.
This patch is a CL for re-landing which was reverted at
http://crrev.com/2080793003.

This CL should be landed after http://crrev.com/2108573002 is committed.

BUG=605844,621788
TEST=./out/Debug/content_unittests
--gtest_filter="ServiceWorkerControlleeRequestHandlerTest.FallbackWithNoFetchHandler*"
==========

[shimazu, 2016-06-28 08:38:23]

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

[shimazu, 2016-06-28 08:38:24]

PTAL at this CL, too.

[horo, 2016-06-29 05:38:30]

https://codereview.chromium.org/2103063002/diff/1/content/browser/service_wor...
File content/browser/service_worker/service_worker_controllee_request_handler.cc
(right):

https://codereview.chromium.org/2103063002/diff/1/content/browser/service_wor...
content/browser/service_worker/service_worker_controllee_request_handler.cc:41:
job->FallbackToNetwork();
I think this should be FallbackToNetworkOrRenderer().

https://codereview.chromium.org/2108573002/diff/1/content/browser/service_wor...

[shimazu, 2016-07-04 07:50:27]

Fixed. PTAL again.

https://codereview.chromium.org/2103063002/diff/1/content/browser/service_wor...
File content/browser/service_worker/service_worker_controllee_request_handler.cc
(right):

https://codereview.chromium.org/2103063002/diff/1/content/browser/service_wor...
content/browser/service_worker/service_worker_controllee_request_handler.cc:41:
job->FallbackToNetwork();
On 2016/06/29 05:38:30, horo wrote:
> I think this should be FallbackToNetworkOrRenderer().
> 
>
https://codereview.chromium.org/2108573002/diff/1/content/browser/service_wor...

Done.

[horo, 2016-07-05 03:00:32]

lgtm

[shimazu, 2016-07-08 04:17:07]

The CQ bit was checked by shimazu@chromium.org

[commit-bot: I haz the power, 2016-07-08 04:17:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-08 14:04:27]

Description was changed from

==========
ServiceWorker: Reland of bypassing SW when no fetch handler existed

When the script doesn't have fetch handler, in-scope network requests
don't need to go to the service worker. This patch is to bypass it.
This patch is a CL for re-landing which was reverted at
http://crrev.com/2080793003.

This CL should be landed after http://crrev.com/2108573002 is committed.

BUG=605844,621788
TEST=./out/Debug/content_unittests
--gtest_filter="ServiceWorkerControlleeRequestHandlerTest.FallbackWithNoFetchHandler*"
==========

to

==========
ServiceWorker: Reland of bypassing SW when no fetch handler existed

When the script doesn't have fetch handler, in-scope network requests
don't need to go to the service worker. This patch is to bypass it.
This patch is a CL for re-landing which was reverted at
http://crrev.com/2080793003.

This CL should be landed after http://crrev.com/2108573002 is committed.

BUG=605844,621788
TEST=./out/Debug/content_unittests
--gtest_filter="ServiceWorkerControlleeRequestHandlerTest.FallbackWithNoFetchHandler*"
==========

[commit-bot: I haz the power, 2016-07-08 14:04:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-08 14:05:04]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-08 14:06:27]

Description was changed from

==========
ServiceWorker: Reland of bypassing SW when no fetch handler existed

When the script doesn't have fetch handler, in-scope network requests
don't need to go to the service worker. This patch is to bypass it.
This patch is a CL for re-landing which was reverted at
http://crrev.com/2080793003.

This CL should be landed after http://crrev.com/2108573002 is committed.

BUG=605844,621788
TEST=./out/Debug/content_unittests
--gtest_filter="ServiceWorkerControlleeRequestHandlerTest.FallbackWithNoFetchHandler*"
==========

to

==========
ServiceWorker: Reland of bypassing SW when no fetch handler existed

When the script doesn't have fetch handler, in-scope network requests
don't need to go to the service worker. This patch is to bypass it.
This patch is a CL for re-landing which was reverted at
http://crrev.com/2080793003.

This CL should be landed after http://crrev.com/2108573002 is committed.

BUG=605844,621788
TEST=./out/Debug/content_unittests
--gtest_filter="ServiceWorkerControlleeRequestHandlerTest.FallbackWithNoFetchHandler*"

Committed: https://crrev.com/cb53dfa21239d8be2ed3bc53c83d8c001603cee6
Cr-Commit-Position: refs/heads/master@{#404348}
==========

[commit-bot: I haz the power, 2016-07-08 14:06:28]

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