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

[bugzilla-daemon at webkit.org]

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


ap at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #27584|review?                     |review+
               Flag|                            |




------- Comment #26 from ap at webkit.org  2009-02-12 00:53 PDT -------
(From update of attachment 27584)
+#include "Console.h"
+#include "ScriptExecutionContext.h"
+#include <wtf/PassRefPtr.h>

Most of these includes do not look necessary in WorkerObjectProxy.h.

+// FIXME: For now, we just cast object proxy pointer in order to access the
functionalities needed for loader.

A FIXME should say what's wrong with the current situation, not just describe
it. To anyone actively working on this code, it's pretty obvious what the
problem is, but it may not be so obvious to others.

A good ChangeLog should say more about the changes - in this case, explaining
that this is a step towards splitting proxies for multi-process design would
suffice.

r=me - this patch looks really good!


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