[Webkit-unassigned] [Bug 23859] Change worker code to use different proxy class pointers.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Feb 10 12:26:59 PST 2009


------- Comment #8 from levin at chromium.org  2009-02-10 12:26 PDT -------
> Worker::hasPendingActivity() 

1. Can this method be called when m_contextProxy is 0 (after the Worker
Have all usages of m_contextProxy been checked to make sure that they won't be
used when it is 0.

> m_unconfirmedMessageCount--; // workerThreadCreated checks if this count is 0.
Now, I understand why you moved it, but I still don't understand if this change
in what WorkerMessagingProxy::hasPendingActivity() returns is ok.  

It looks like there is a variable "m_workerThreadHadPendingActivity" to
indicate that the worker has pending activity.  Perhaps it should be used
instead of the message count in this case.

1. Extra blank lines added to the end.

>   static WorkerContextProxy* create(const String& scriptURL, Worker* worker);
1. remove "worker". It doesn't add any info.

1. Extra blank lines added to the end.

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.

More information about the webkit-unassigned mailing list