[webkit-reviews] review granted: [Bug 12160] Linux/Gdk build fixes
: [Attachment 12995] Linux\gdk build fixes - el patch numero siete
bugzilla-request-daemon at macosforge.org
bugzilla-request-daemon at macosforge.org
Wed Feb 7 02:03:39 PST 2007
Maciej Stachowiak <mjs at apple.com> has granted Maciej Stachowiak
<mjs at apple.com>'s request for review:
Bug 12160: Linux/Gdk build fixes
http://bugs.webkit.org/show_bug.cgi?id=12160
Attachment 12995: Linux\gdk build fixes - el patch numero siete
http://bugs.webkit.org/attachment.cgi?id=12995&action=edit
------- Additional Comments from Maciej Stachowiak <mjs at apple.com>
I'm not sure ResourceHandleManager::self() is a good name. get() was kind of
vague, but a more typical name for a static method when applying the singleton
pattern like this would be sharedInstance() or just shared(). Otherwise these
changes look fine. r=me and feel free to adjust name when landing if you want.
More information about the webkit-reviews
mailing list