RESOLVED FIXED 28136
Need to refactor WorkerObjectProxy 
https://bugs.webkit.org/show_bug.cgi?id=28136
Summary Need to refactor WorkerObjectProxy
Andrew Wilson
Reported 2009-08-09 21:34:00 PDT
Some of the APIs in WorkerObjectProxy are also useful for SharedWorkers - we should refactor the common APIs into a separate class.
Attachments
proposed patch (28.08 KB, patch)
2009-08-09 21:52 PDT, Andrew Wilson 		levin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andrew Wilson
Comment 1 2009-08-09 21:52:40 PDT
Created attachment 34439 [details] proposed patch This is the first part of my upcoming patch that adds lifecycle management for shared workers.
David Levin
Comment 2 2009-08-11 02:03:19 PDT
Comment on attachment 34439 [details] proposed patch > diff --git a/WebCore/workers/WorkerReportingProxy.h b/WebCore/workers/WorkerReportingProxy.h > + > + // APIs used by workers to report console activity. > + class WorkerReportingProxy { > + public: > + virtual ~WorkerReportingProxy() {} Nice to add a space in the {}
Andrew Wilson
Comment 3 2009-08-11 09:33:47 PDT
Committed as r47036
Note You need to log in before you can comment on or make changes to this bug.