Bug 22861 – Turn the FontCache into a singleton

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Julien Chaffraix]

As discussed on the mailing list, this is part of the initial clean-up for the fontcache.

Patch forthcoming.

[Julien Chaffraix]

Created attachment 26020 [details]
Proposed fix

[Sam Weinig]

Why is it a good idea to turn the FontCache into a singleton?  What does this get us?

[Julien Chaffraix]

(In reply to comment #2)
> Why is it a good idea to turn the FontCache into a singleton?  What does this
> get us?
> 

First it makes FontCache coherent with the rest of WebCore: I have not found another class that only bears static methods inside WebCore. IMHO it is also particularly relevant to make this class close to the Cache.
Then it makes it cristal clear that we are using a singleton: static methods + static HashMaps / HashList inside the .cpp sounds like a hidden singleton.
Finally when you start hacking the FontCache as it is now, you have to find the different definitions mixed with the code in FontCache.cpp. Making it a real class is a first step into moving those definitons in their own files and move the static HashMaps / HashList in the header.

So to answer the benefits: right now, it adds a little overhead to the compiled code. But within a few changes, we should have a more readable code.

[Julien Chaffraix]

Created attachment 26065 [details]
Same as the first version, with corrected ChangeLogs

I had forgotten to update the ChangeLog in the WebKit directory, so attached is an updated version.

[Darin Adler]

Comment on attachment 26065 [details]
Same as the first version, with corrected ChangeLogs

I'm not entirely convinced this is an improvement, but it seems OK. r=me

[Julien Chaffraix]

Landed patch in r39823.