[Webkit-unassigned] [Bug 26629] Refactor localStorage code for use in multi-process browsers (part 2)

[bugzilla-daemon at webkit.org]

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


beidson at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Simplify LocalStorage code  |Refactor localStorage code
                   |and make initial import     |for use in multi-process
                   |related semantics more      |browsers (part 2)
                   |conservitive/correct        |




------- Comment #5 from beidson at apple.com  2009-06-22 18:03 PDT -------
I see now that the expected optimization with set/remove/clear was never
actually there.  The code was complicated like this for that very reason.

getItem() was the one I actually already knew there was a flaw with  ;)

My concern stands in a new form - changing the fine grain code to a big heavy
block(), it seems to me the fine grained code will just have to be
re-introduced at some point to get closer to the original intended
optimizations.

Since they aren't in place, though, I guess the patch is fine.


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