[Webkit-unassigned] [Bug 24054] Implement two separate proxy classes in order to split WorkerMessagingProxy.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 19 23:45:40 PST 2009


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





------- Comment #3 from dimich at chromium.org  2009-02-19 23:45 PDT -------
Small naming comment: looking at this patch, I think WorkerObjectProxy and
WorkerContextProxy have some names unnecessarily repeated. 

WorkerObjectCrossThreadProxy has some methods that are named
xxxxxToWorkerObject and
WorkerContextCrossThreadProxy has some methods that are named
xxxxxToWorkerContext.

It seems the 'ToWorkerObject' and 'ToWorkerContext' are unnecessary in method
names since this is already implied in the naming of proxies themselves. It
probably deserves a separate patch though.


-- 
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