[webkit-reviews] review granted: [Bug 121100] [Windows] Revise GDI Create Functions to use GDIObject Smart Pointer : [Attachment 211341] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Sep 11 12:48:01 PDT 2013


Anders Carlsson <andersca at apple.com> has granted Brent Fulgham
<bfulgham at webkit.org>'s request for review:
Bug 121100: [Windows] Revise GDI Create Functions to use GDIObject Smart
Pointer
https://bugs.webkit.org/show_bug.cgi?id=121100

Attachment 211341: Patch
https://bugs.webkit.org/attachment.cgi?id=211341&action=review

------- Additional Comments from Anders Carlsson <andersca at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=211341&action=review


> Source/WebCore/platform/graphics/win/FontCacheWin.cpp:491
> +	   return GDIObject<HFONT>();

If you add a GDIObject constructor that takes std::nullptr_t you can just
return nullptr here.

> Source/WebCore/platform/graphics/win/FontPlatformDataWin.cpp:58
> +    ::SelectObject(hdc, m_font.get()->get());

I think SharedGDIObject’s get function should just return T, then this would
just be m_font.get().

> Source/WebCore/platform/graphics/win/FontPlatformDataWin.cpp:69
> +	   platformDataInit(m_font.get()->get(), size, hdc, faceName);

Same thing here.


More information about the webkit-reviews mailing list