[webkit-dev] JavaScript bindings changed to call scriptExecutionContext virtual function?
Darin Adler
darin at apple.com
Sun Aug 2 02:09:17 PDT 2009
On Aug 2, 2009, at 2:05 AM, Adam Barth wrote:
> In any case:
>
> https://bugs.webkit.org/show_bug.cgi?id=27931
>
> I'll have a patch shortly. Please let me know if there are other
> call sites you'd like changed.
Thanks! Once you pointed out this was not due to a recent change, I
had resolved to fix it myself, but I’m glad you’re going to tackle it.
Generally speaking no call site that has a Node* should call
scriptExecutionContext(). So rather than listing the call sites I want
changed, I’d instead suggest that we make Node::scriptExecutionContext
() private and then make sure no Node member functions or members
functions of friend classes call the function.
And the same is true of most other classes derived from EventTarget.
-- Darin
More information about the webkit-dev
mailing list