Revert of Implement dumb URLLoader{Factory} for ServiceWorker script loading

Revert of Implement dumb URLLoader{Factory} for ServiceWorker script loading (patchset #4 id:60001 of https://codereview.chromium.org/2943463002/ )

Reason for revert:
This patch depends on https://crrev.com/1ebf3cb which is being reverted. 
See also the bug: https://crbug.com/736649

Original issue's description:
> Implement dumb URLLoader{Factory} for ServiceWorker script loading
> 
> Currently it only proxies request/response to/from the network factory,
> plus does necessary HttpResponseInfo passing.
> 
> The plan is to use this URLLoader for script loading/caching during
> installation. (Currently this always sets-up the URLLoader{Factory}
> if servicification is enabled because we need it for correctly setting
> HttpResponseInfo of the main resource of ServiceWorker)
> 
> (Patch on top of https://codereview.chromium.org/2779763004/
> Try: https://codereview.chromium.org/2936623002/)
> 
> BUG=715640
> 
> Review-Url: https://codereview.chromium.org/2943463002
> Cr-Commit-Position: refs/heads/master@{#481477}
> Committed: https://chromium.googlesource.com/chromium/src/+/e7ab19876d2a092ef09f584619ea82890e89050e

TBR=falken@chromium.org,dcheng@chromium.org,kinuko@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=715640, 736649
Review-Url: https://codereview.chromium.org/2959903002
Cr-Commit-Position: refs/heads/master@{#482554}
Committed: https://chromium.googlesource.com/chromium/src/+/751f0a2995bb1975449572a0b5328ec4db48b762

[shimazu, 2017-06-27 04:13:12]

The CQ bit was checked by shimazu@chromium.org

[shimazu, 2017-06-27 04:13:13]

Created Revert of Implement dumb URLLoader{Factory} for ServiceWorker script
loading

[commit-bot: I haz the power, 2017-06-27 04:13:23]

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, 2017-06-27 04:15:35]

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

[commit-bot: I haz the power, 2017-06-27 04:15:36]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[shimazu, 2017-06-27 04:23:07]

The CQ bit was checked by shimazu@chromium.org

[commit-bot: I haz the power, 2017-06-27 04:23:24]

CQ is trying da patch.

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

[shimazu, 2017-06-27 04:23:29]

Description was changed from

==========
Revert of Implement dumb URLLoader{Factory} for ServiceWorker script loading
(patchset #4 id:60001 of https://codereview.chromium.org/2943463002/ )

Reason for revert:
This patch depends on https://crrev.com/1ebf3cb which is being reverted. 
See also the bug: https://crbug.com/736649

Original issue's description:
> Implement dumb URLLoader{Factory} for ServiceWorker script loading
> 
> Currently it only proxies request/response to/from the network factory,
> plus does necessary HttpResponseInfo passing.
> 
> The plan is to use this URLLoader for script loading/caching during
> installation. (Currently this always sets-up the URLLoader{Factory}
> if servicification is enabled because we need it for correctly setting
> HttpResponseInfo of the main resource of ServiceWorker)
> 
> (Patch on top of https://codereview.chromium.org/2779763004/
> Try: https://codereview.chromium.org/2936623002/)
> 
> BUG=715640
> 
> Review-Url: https://codereview.chromium.org/2943463002
> Cr-Commit-Position: refs/heads/master@{#481477}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7ab19876d2a092ef09f584619ea...

TBR=falken@chromium.org,dcheng@chromium.org,kinuko@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=715640
==========

to

==========
Revert of Implement dumb URLLoader{Factory} for ServiceWorker script loading
(patchset #4 id:60001 of https://codereview.chromium.org/2943463002/ )

Reason for revert:
This patch depends on https://crrev.com/1ebf3cb which is being reverted. 
See also the bug: https://crbug.com/736649

Original issue's description:
> Implement dumb URLLoader{Factory} for ServiceWorker script loading
> 
> Currently it only proxies request/response to/from the network factory,
> plus does necessary HttpResponseInfo passing.
> 
> The plan is to use this URLLoader for script loading/caching during
> installation. (Currently this always sets-up the URLLoader{Factory}
> if servicification is enabled because we need it for correctly setting
> HttpResponseInfo of the main resource of ServiceWorker)
> 
> (Patch on top of https://codereview.chromium.org/2779763004/
> Try: https://codereview.chromium.org/2936623002/)
> 
> BUG=715640
> 
> Review-Url: https://codereview.chromium.org/2943463002
> Cr-Commit-Position: refs/heads/master@{#481477}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7ab19876d2a092ef09f584619ea...

TBR=falken@chromium.org,dcheng@chromium.org,kinuko@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=715640 736649
==========

[shimazu, 2017-06-27 04:23:43]

Description was changed from

==========
Revert of Implement dumb URLLoader{Factory} for ServiceWorker script loading
(patchset #4 id:60001 of https://codereview.chromium.org/2943463002/ )

Reason for revert:
This patch depends on https://crrev.com/1ebf3cb which is being reverted. 
See also the bug: https://crbug.com/736649

Original issue's description:
> Implement dumb URLLoader{Factory} for ServiceWorker script loading
> 
> Currently it only proxies request/response to/from the network factory,
> plus does necessary HttpResponseInfo passing.
> 
> The plan is to use this URLLoader for script loading/caching during
> installation. (Currently this always sets-up the URLLoader{Factory}
> if servicification is enabled because we need it for correctly setting
> HttpResponseInfo of the main resource of ServiceWorker)
> 
> (Patch on top of https://codereview.chromium.org/2779763004/
> Try: https://codereview.chromium.org/2936623002/)
> 
> BUG=715640
> 
> Review-Url: https://codereview.chromium.org/2943463002
> Cr-Commit-Position: refs/heads/master@{#481477}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7ab19876d2a092ef09f584619ea...

TBR=falken@chromium.org,dcheng@chromium.org,kinuko@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=715640 736649
==========

to

==========
Revert of Implement dumb URLLoader{Factory} for ServiceWorker script loading
(patchset #4 id:60001 of https://codereview.chromium.org/2943463002/ )

Reason for revert:
This patch depends on https://crrev.com/1ebf3cb which is being reverted. 
See also the bug: https://crbug.com/736649

Original issue's description:
> Implement dumb URLLoader{Factory} for ServiceWorker script loading
> 
> Currently it only proxies request/response to/from the network factory,
> plus does necessary HttpResponseInfo passing.
> 
> The plan is to use this URLLoader for script loading/caching during
> installation. (Currently this always sets-up the URLLoader{Factory}
> if servicification is enabled because we need it for correctly setting
> HttpResponseInfo of the main resource of ServiceWorker)
> 
> (Patch on top of https://codereview.chromium.org/2779763004/
> Try: https://codereview.chromium.org/2936623002/)
> 
> BUG=715640
> 
> Review-Url: https://codereview.chromium.org/2943463002
> Cr-Commit-Position: refs/heads/master@{#481477}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7ab19876d2a092ef09f584619ea...

TBR=falken@chromium.org,dcheng@chromium.org,kinuko@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=715640, 736649
==========

[commit-bot: I haz the power, 2017-06-27 04:25:34]

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

[commit-bot: I haz the power, 2017-06-27 04:25:34]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[shimazu, 2017-06-27 04:30:57]

Patchset #2 (id:310001) has been deleted

[shimazu, 2017-06-27 04:33:07]

The CQ bit was checked by shimazu@chromium.org

[commit-bot: I haz the power, 2017-06-27 04:33:15]

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, 2017-06-27 06:53:57]

CQ is committing da patch.

Bot data: {"patchset_id": 20002, "attempt_start_ts": 1498537987094990,
"parent_rev": "fcd93adf4cf79f45b519eb10b46f60ee7097bf2c", "commit_rev":
"751f0a2995bb1975449572a0b5328ec4db48b762"}

[commit-bot: I haz the power, 2017-06-27 06:54:11]

Description was changed from

==========
Revert of Implement dumb URLLoader{Factory} for ServiceWorker script loading
(patchset #4 id:60001 of https://codereview.chromium.org/2943463002/ )

Reason for revert:
This patch depends on https://crrev.com/1ebf3cb which is being reverted. 
See also the bug: https://crbug.com/736649

Original issue's description:
> Implement dumb URLLoader{Factory} for ServiceWorker script loading
> 
> Currently it only proxies request/response to/from the network factory,
> plus does necessary HttpResponseInfo passing.
> 
> The plan is to use this URLLoader for script loading/caching during
> installation. (Currently this always sets-up the URLLoader{Factory}
> if servicification is enabled because we need it for correctly setting
> HttpResponseInfo of the main resource of ServiceWorker)
> 
> (Patch on top of https://codereview.chromium.org/2779763004/
> Try: https://codereview.chromium.org/2936623002/)
> 
> BUG=715640
> 
> Review-Url: https://codereview.chromium.org/2943463002
> Cr-Commit-Position: refs/heads/master@{#481477}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7ab19876d2a092ef09f584619ea...

TBR=falken@chromium.org,dcheng@chromium.org,kinuko@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=715640, 736649
==========

to

==========
Revert of Implement dumb URLLoader{Factory} for ServiceWorker script loading
(patchset #4 id:60001 of https://codereview.chromium.org/2943463002/ )

Reason for revert:
This patch depends on https://crrev.com/1ebf3cb which is being reverted. 
See also the bug: https://crbug.com/736649

Original issue's description:
> Implement dumb URLLoader{Factory} for ServiceWorker script loading
> 
> Currently it only proxies request/response to/from the network factory,
> plus does necessary HttpResponseInfo passing.
> 
> The plan is to use this URLLoader for script loading/caching during
> installation. (Currently this always sets-up the URLLoader{Factory}
> if servicification is enabled because we need it for correctly setting
> HttpResponseInfo of the main resource of ServiceWorker)
> 
> (Patch on top of https://codereview.chromium.org/2779763004/
> Try: https://codereview.chromium.org/2936623002/)
> 
> BUG=715640
> 
> Review-Url: https://codereview.chromium.org/2943463002
> Cr-Commit-Position: refs/heads/master@{#481477}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e7ab19876d2a092ef09f584619ea...

TBR=falken@chromium.org,dcheng@chromium.org,kinuko@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=715640, 736649

Review-Url: https://codereview.chromium.org/2959903002
Cr-Commit-Position: refs/heads/master@{#482554}
Committed:
https://chromium.googlesource.com/chromium/src/+/751f0a2995bb1975449572a0b532...
==========

[commit-bot: I haz the power, 2017-06-27 06:54:13]

Committed patchset #2 (id:20002) as
https://chromium.googlesource.com/chromium/src/+/751f0a2995bb1975449572a0b532...