[Webkit-unassigned] [Bug 24055] Replace WorkerMessagingProxy with two separate proxy classes.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=24055





------- Comment #5 from levin at chromium.org  2009-02-20 16:14 PDT -------
This looks good to me.

There may be a lifetime issue with m_workerContext in usage in the bridge code,
but I'll examine that more later as I get up tests for xhr in workers.  Also as
mentioned before I think we can figure out a way to make lifetime easier to
understand overall.


> > +void Worker::reportPendingActivity(bool confirmMessage, bool hasPendingActivity)
> WebKit eng is trying to move away from bool's to enums.  I think that would be
> great here b/c I've found places calling this to be unreadable.

It would be good to do this at some point in the future.


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.