The current name is vague and confusing.
Created attachment 244061 [details] patch
Comment on attachment 244061 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=244061&action=review > Source/WebCore/platform/graphics/SimpleFontData.h:179 > + bool isSVGFont() const { return !!m_svgData; } Is the !! necessary? Won't the bool cast be automatic?
Comment on attachment 244061 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=244061&action=review >> Source/WebCore/platform/graphics/SimpleFontData.h:179 >> + bool isSVGFont() const { return !!m_svgData; } > > Is the !! necessary? Won't the bool cast be automatic? It won't since the std::unique_ptr bool operator is explicit.
https://trac.webkit.org/r177975