[ServiceWorker] Introduce isControlledByServiceWorker() in content::RenderFrameImpl

[ServiceWorker] Introduce isControlledByServiceWorker() in content::RenderFrameImpl

We need to check this in DocumentThreadableLoader not to send CORS preflight requests to the ServiceWorker.

Blink side is implemented in https://codereview.chromium.org/546263002/.


BUG=411151
Committed: https://crrev.com/96d38dadb4ef427e56542c7316b755d47de680e4
Cr-Commit-Position: refs/heads/master@{#294156}

[horo, 2014-09-08 04:57:29]

Patchset #1 (id:1) has been deleted

[horo, 2014-09-08 05:05:15]

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

[horo, 2014-09-08 05:05:16]

nhiroki@
Could you please review this?

[nhiroki, 2014-09-08 05:31:19]

LGTM from ServiceWorker's POV

[nhiroki, 2014-09-08 05:45:34]

I forgot to publish a comment...

https://codereview.chromium.org/553603002/diff/20001/content/renderer/render_...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/553603002/diff/20001/content/renderer/render_...
content/renderer/render_frame_impl.cc:3205: kInvalidServiceWorkerHandleId;
nit: Maybe we need (1) -3-space indent or (2) +4-space indent.

[horo, 2014-09-08 06:24:25]

https://codereview.chromium.org/553603002/diff/20001/content/renderer/render_...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/553603002/diff/20001/content/renderer/render_...
content/renderer/render_frame_impl.cc:3205: kInvalidServiceWorkerHandleId;
On 2014/09/08 05:45:33, nhiroki wrote:
> nit: Maybe we need (1) -3-space indent or (2) +4-space indent.

Done.

[horo, 2014-09-08 08:34:22]

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

[horo, 2014-09-08 08:34:22]

jochen@
Could you please review this?

[jochen (gone - plz use gerrit), 2014-09-10 08:15:33]

lgtm

[horo, 2014-09-10 08:38:51]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-09-10 08:39:39]

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

[commit-bot: I haz the power, 2014-09-10 10:33:06]

Committed patchset #3 (id:60001) as c2c0f2be2c9f5566e2bb10ae43e6f3aa8e9eb918

[commit-bot: I haz the power, 2014-09-10 10:56:29]

Patchset 3 (id:??) landed as
https://crrev.com/96d38dadb4ef427e56542c7316b755d47de680e4
Cr-Commit-Position: refs/heads/master@{#294156}