[Webkit-unassigned] [Bug 98541] Switch CachedResource over from SharedBuffer to a new ResourceBuffer

[bugzilla-daemon at webkit.org]

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





--- Comment #4 from Brady Eidson <beidson at apple.com>  2012-10-05 15:00:50 PST ---
(In reply to comment #3)
> I guess I'm interested in the bigger picture of what changes you'll be making to WebCore in this area.  In particular, I'd like a chance to convince you that a shared memory cache is overengineering.  :)

Some refactoring to WebCore boils down to almost direct replacement-style refactoring and should bother few folks, if any.  Once such patch should be coming to this bug soon and I have a few others in the pipeline right behind it.

There's almost certainly going to have to be some sort of structural changes to the CachedResource/ResourceLoader family of classes.  I think everyone can agree that the WebCore/loader directory is a mess of code and needs some love anyways.  So it follows we intend for such refactoring to be an opportunity to clean the code, not just shoehorn in what we need.

As far as convincing us a shared memory cache is over-engineering...  we simply have data that shows it's a win with some basic process models.

If you have data that shows it is *NOT* a win, that'd be super interesting.

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