Move ValueToString out to its own header file to remove duplication
Created attachment 216732 [details] Patch
Comment on attachment 216732 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=216732&action=review r=me > Source/WebCore/platform/LayoutUnit.h:34 > +#ifndef NDEBUG nit: You don't need the #ifndef here. The file itself already has it. Also, you don't do that in the other headers. > Source/WebCore/platform/LayoutUnit.h:970 > +template <> struct ValueToString<LayoutUnit> { nit: you have different styles for the template<> placement. I think the second one is better. template <> struct ValueToString<LayoutUnit> and template <> struct ValueToString<LayoutUnit>
Created attachment 216735 [details] Updated Patch
Comment on attachment 216735 [details] Updated Patch Clearing flags on attachment: 216735 Committed r159159: <http://trac.webkit.org/changeset/159159>
All reviewed patches have been landed. Closing bug.