Refactor SharedWorkerHost and SharedWorkerInstance.

Refactor SharedWorkerHost and SharedWorkerInstance.

In this CL, I move the status variables of SharedWorker from SharedWorkerInstance to SharedWorkerHost.

The main purpose of this change is to make it possible to pass SharedWorkerInstance to SharedWorkerDevToolsManager and store it in WorkerInfo in http://crrev.com/196503005.

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

[horo, 2014-03-27 07:44:07]

kinuko@
Could you please review?

[kinuko, 2014-03-28 11:12:49]

looks good

Is this basically for cleanup?  Could you tell the main purpose of this CL?

[horo, 2014-03-29 00:37:22]

On 2014/03/28 11:12:49, kinuko wrote:
> looks good
> 
> Is this basically for cleanup?  Could you tell the main purpose of this CL?

The main purpose is to make it possible to pass SharedWorkerInstance to
SharedWorkerDevToolsManager and store it in WorkerInfo in
http://crrev.com/196503005.

Should I this cleanup in http://crrev.com/196503005 ?

[kinuko, 2014-03-31 00:59:32]

On 2014/03/29 00:37:22, horo wrote:
> On 2014/03/28 11:12:49, kinuko wrote:
> > looks good
> > 
> > Is this basically for cleanup?  Could you tell the main purpose of this CL?
> 
> The main purpose is to make it possible to pass SharedWorkerInstance to
> SharedWorkerDevToolsManager and store it in WorkerInfo in
> http://crrev.com/196503005.
> 
> Should I this cleanup in http://crrev.com/196503005 ?

Nope, I just wanted to have more contexts (this CL itself doesn't tell much). 
It'd have been nicer if you made your intention clear in the CL description etc.

lgtm

[horo, 2014-03-31 01:08:45]

On 2014/03/31 00:59:32, kinuko wrote:
> On 2014/03/29 00:37:22, horo wrote:
> > On 2014/03/28 11:12:49, kinuko wrote:
> > > looks good
> > > 
> > > Is this basically for cleanup?  Could you tell the main purpose of this
CL?
> > 
> > The main purpose is to make it possible to pass SharedWorkerInstance to
> > SharedWorkerDevToolsManager and store it in WorkerInfo in
> > http://crrev.com/196503005.
> > 
> > Should I this cleanup in http://crrev.com/196503005 ?
> 
> Nope, I just wanted to have more contexts (this CL itself doesn't tell much). 
> It'd have been nicer if you made your intention clear in the CL description
etc.
> 
> lgtm

Added to the CL's description.

[horo, 2014-03-31 01:08:49]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-03-31 01:08:58]

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

[commit-bot: I haz the power, 2014-03-31 03:20:35]

Change committed as 260489