Set SkipServiceWorker flag in DocumentThreadableLoader::loadActualRequest().

Set SkipServiceWorker flag in DocumentThreadableLoader::loadActualRequest().

Before a SW controls the page, when the DTL sends a CORS preflight request, the SkipServiceWorker flag is set by RenderFrameImpl::willSendRequest().
But a new SW may be controlling the page when the DTL sends the actual request by calling clients.claim().
In such case, the request goes to the SW and causes several problems.

To avoid this problem, this cl set the flag in DTL::loadActualRequest().

BUG=604583, 610400
Review-Url: https://codereview.chromium.org/1964823002
Cr-Commit-Position: refs/heads/master@{#392609}
(cherry picked from commit 01fb3c2cf2bf84391c488f3a3dd5ef8c59cc0a51)

Committed: https://chromium.googlesource.com/chromium/src/+/b083560ed010859fb4cdd23626ddef597d40b1f4

[horo, 2016-05-12 02:19:49]

Description was changed from

==========
Set SkipServiceWorker flag in DocumentThreadableLoader::loadActualRequest().

Before a SW controls the page, when the DTL sends a CORS preflight request, the
SkipServiceWorker flag is set by RenderFrameImpl::willSendRequest().
But a new SW may be controlling the page when the DTL sends the actual request
by calling clients.claim().
In such case, the request goes to the SW and causes several problems.

To avoid this problem, this cl set the flag in DTL::loadActualRequest().

BUG=604583,610400

Review-Url: https://codereview.chromium.org/1964823002
Cr-Commit-Position: refs/heads/master@{#392609}
(cherry picked from commit 01fb3c2cf2bf84391c488f3a3dd5ef8c59cc0a51)
==========

to

==========
Set SkipServiceWorker flag in DocumentThreadableLoader::loadActualRequest().

Before a SW controls the page, when the DTL sends a CORS preflight request, the
SkipServiceWorker flag is set by RenderFrameImpl::willSendRequest().
But a new SW may be controlling the page when the DTL sends the actual request
by calling clients.claim().
In such case, the request goes to the SW and causes several problems.

To avoid this problem, this cl set the flag in DTL::loadActualRequest().

BUG=604583,610400

Review-Url: https://codereview.chromium.org/1964823002
Cr-Commit-Position: refs/heads/master@{#392609}
(cherry picked from commit 01fb3c2cf2bf84391c488f3a3dd5ef8c59cc0a51)

Committed:
https://chromium.googlesource.com/chromium/src/+/b083560ed010859fb4cdd23626dd...
==========

[horo, 2016-05-12 02:19:51]

Committed patchset #1 (id:1) manually as
b083560ed010859fb4cdd23626ddef597d40b1f4 (tree was closed).