[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