[Webkit-unassigned] [Bug 62653] [V8][Chromium] Make StringCache in V8 bindings per-isolate
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Jun 16 05:56:51 PDT 2011
https://bugs.webkit.org/show_bug.cgi?id=62653
--- Comment #15 from anton muhin <antonm at chromium.org> 2011-06-16 05:56:51 PST ---
Overall, I still have concerns about approach taken.
Moving all this stuff makes things more complex. And most probably slower---even if each separate patch makes as slower by ~1% which is indistinguishable from noise, we may die of death of million cuts.
I would really prefer if there was a separate branch with the full implementation and if overall complexity and perf is good enough, we'll start to bring the changes to WebCore.
I would also appreciate more thorough discussion of different approaches, their benefits and drawbacks. For example, Vitaly (p.c.) noted that we probably do not need to make all FunctionTenplate into per-isolate data as the set of ones used in Workers vs. ones in DOM pages has minimal (if any) intersection.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list