Bug 90317
| Summary: | [V8] Use WebCore::root for V8 bindings too | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Erik Arvidsson <arv> |
| Component: | DOM | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | haraken |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Erik Arvidsson
I've been working on GenerateIsReachable related stuff and it seems like we should use WebCore::root too. I think we can get rid of all non Node cases by delegating to the WebCore::root value.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Kentaro Hara
(In reply to comment #0)
> it seems like we should use WebCore::root too. I think we can get rid of all non Node cases by delegating to the WebCore::root value.
I would guess root() is not a WebCore method but a method implemented in JSDOMBinding.h. We might want to move it to Node.h so that both JSC and V8 can use it.
> I've been working on GenerateIsReachable related stuff
Just in case, would you elaborate on the overall goal of the work?
Erik Arvidsson
(In reply to comment #1)
> (In reply to comment #0)
> > it seems like we should use WebCore::root too. I think we can get rid of all non Node cases by delegating to the WebCore::root value.
>
> I would guess root() is not a WebCore method but a method implemented in JSDOMBinding.h. We might want to move it to Node.h so that both JSC and V8 can use it.
Yeah, right now these are only used by the JSC bindings. I was thinking to put them in Source/WebCore/bindings/.
Node.h is not the right place for these. Some of them are unrelated to Nodes (WebCore::root(StyleSheet*) for example).
> Just in case, would you elaborate on the overall goal of the work?
There are two goals here.
1. Correctness. Reachable wrappers should not be collected.
2. Security. If GC is observable it can be used as a covert channel.
Brian Burg
Closing some V8-related work items.