Add commandline flag for ServiceWroker in the renderer process.

Add commandline flag "enable-embedded-shared-worker" for ServiceWroker in the renderer process.

This will allow the SharedWorker to run inside the renderer process instead of the worker process.

BUG=327256
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=256421

[horo, 2014-02-10 08:56:01]

kinuko@
Could you please review this CL?

[kinuko, 2014-02-10 09:00:00]

lgtm

[horo, 2014-02-10 09:04:46]

jochen@
Please review this cl.
Thank you.

[jochen (gone - plz use gerrit), 2014-02-10 12:23:12]

Is it possible to add the flag together with the code that actually uses it?

e.g. right now, I can't see why you need to export the flag

[horo, 2014-02-12 01:48:48]

On 2014/02/10 12:23:12, jochen wrote:
> Is it possible to add the flag together with the code that actually uses it?
> 
> e.g. right now, I can't see why you need to export the flag

Ok.
I will add this flag when it will be needed.
Thank you.

[horo, 2014-03-11 06:21:27]

kinuko@
I added the following files which use this flag.
 content/browser/renderer_host/render_process_host_impl.cc
 content/browser/worker_host/worker_service_impl.cc
 content/public/browser/worker_service.h


Could you please review this cl again?
Thank you.

[kinuko, 2014-03-11 07:40:19]

lgtm

https://codereview.chromium.org/154263004/diff/90001/content/browser/worker_h...
File content/browser/worker_host/worker_service_impl.cc (right):

https://codereview.chromium.org/154263004/diff/90001/content/browser/worker_h...
content/browser/worker_host/worker_service_impl.cc:236: }
nit: no need of { } for single-line body

[horo, 2014-03-11 07:49:19]

https://codereview.chromium.org/154263004/diff/90001/content/browser/worker_h...
File content/browser/worker_host/worker_service_impl.cc (right):

https://codereview.chromium.org/154263004/diff/90001/content/browser/worker_h...
content/browser/worker_host/worker_service_impl.cc:236: }
On 2014/03/11 07:40:19, kinuko wrote:
> nit: no need of { } for single-line body

Done.

[horo, 2014-03-11 07:51:23]

jochen@
Could you please review this cl again?
I added the following files which use this flag.
 content/browser/renderer_host/render_process_host_impl.cc
 content/browser/worker_host/worker_service_impl.cc
 content/public/browser/worker_service.h

Thank you.

[jochen (gone - plz use gerrit), 2014-03-11 14:33:45]

lgtm

[horo, 2014-03-11 15:25:27]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-03-11 15:26:53]

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

[commit-bot: I haz the power, 2014-03-11 20:29:52]

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

[commit-bot: I haz the power, 2014-03-11 20:29:52]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[horo, 2014-03-11 22:42:25]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-03-11 22:48:15]

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

[commit-bot: I haz the power, 2014-03-12 04:35:38]

Change committed as 256421