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.
(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?
(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.
Closing some V8-related work items.