[Webkit-unassigned] [Bug 32430] [v8, dom] Add a pointer field to C++ Node class

[bugzilla-daemon at webkit.org]

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





--- Comment #17 from anton muhin <antonm at chromium.org>  2009-12-16 09:57:39 PST ---
(In reply to comment #15)
> (In reply to comment #14)
> > (In reply to comment #13)
> > > (In reply to comment #7)
> > > > This bug is quite devoid of any actual details of the speedup you got.  What
> > > > was the win?  What did you test on?  Why is this the right tradeoff?
> > > 
> > > Sorry, Sam, didn't want to overload with details.
> > > 
> > > The speedup is ~10% (tested on Windows and Linux) for overall score for DOM
> > > core of Dromaeo.
> > 
> > At what cost?  What was the change in memory use?
> 
> Another question: Is putting this pointer on Node really the right choice? We
> have discussed specializing Element or  even HTMLElement to avoid the cost for
> every Node (Text nodes for instance). Did you measure which type of Nodes this
> would be the biggest win for?

Sorry, could you elaborate?  Was it discussed in public?  If yes, could you
give me a pointer to the thread?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.