De-bloat SVGPathStringBuilder

Source/wtf/text/StringBuilderTest.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 17 matching lines @@
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 
 #include "wtf/Assertions.h"
 #include "wtf/text/CString.h"
 #include "wtf/text/StringBuilder.h"
 #include "wtf/text/WTFString.h"
+#include "wtf/unicode/CharacterNames.h"
 #include <gtest/gtest.h>
 
 namespace WTF {
 
 static std::ostream& operator<<(std::ostream& os, const String& string)
 {
     return os << string.utf8().data();
 }
 
 }
@@ skipping 261 matching lines @@
     }
     { // With capacity.
         StringBuilder builder;
         builder.reserveCapacity(64);
         builder.append("abc");
         String substring = builder.substring(2, 10);
         ASSERT_EQ(String("c"), substring);
     }
 }
 
+TEST(StringBuilderTest, AppendNumberDoubleUChar)
+{
+    const double someNumber = 1.2345;

[Erik Dahlström (inactive), 2014/03/27 14:42:34]

String::number() uses precision=6 per default, and appendNumber too, but would
this really catch a change in one of the places?

Why not use a const for setting the default value to the same in both places?

Would it make sense to have that as a test too?

+    StringBuilder reference;
+    reference.append(replacementCharacter); // Make it UTF-16.
+    reference.append(String::number(someNumber));
+    StringBuilder test;
+    test.append(replacementCharacter);
+    test.appendNumber(someNumber);
+    ASSERT_EQ(reference, test);
+}
+
 } // namespace