Update the stringifier behavior for DOMMatrixReadOnly

third_party/WebKit/Source/core/geometry/DOMMatrixReadOnly.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/geometry/DOMMatrixReadOnly.h"
 
 #include "bindings/core/v8/V8ObjectBuilder.h"
 #include "core/css/CSSIdentifierValue.h"
 #include "core/css/CSSToLengthConversionData.h"
 #include "core/css/CSSValueList.h"
@@ skipping 286 matching lines @@
   double array[] = {
       matrix_->M11(), matrix_->M12(), matrix_->M13(), matrix_->M14(),
       matrix_->M21(), matrix_->M22(), matrix_->M23(), matrix_->M24(),
       matrix_->M31(), matrix_->M32(), matrix_->M33(), matrix_->M34(),
       matrix_->M41(), matrix_->M42(), matrix_->M43(), matrix_->M44()};
 
   return NotShared<DOMFloat64Array>(DOMFloat64Array::Create(array, 16));
 }
 
 const String DOMMatrixReadOnly::toString() const {
-  std::stringstream stream;
+  String result;
   if (is2D()) {
-    stream << "matrix(" << a() << ", " << b() << ", " << c() << ", " << d()

[zino, 2017/05/09 09:32:04]

Looks good to me.

BTW, don't we use stringstream like previous patch?

stream << "matrix(" <<String::NumberToStringECMAScript(a()) << ", "
...

-           << ", " << e() << ", " << f();
+    result.append("matrix(");
+    result.append(String::NumberToStringECMAScript(a()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(b()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(c()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(d()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(e()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(f()));
   } else {
-    stream << "matrix3d(" << m11() << ", " << m12() << ", " << m13() << ", "
-           << m14() << ", " << m21() << ", " << m22() << ", " << m23() << ", "
-           << m24() << ", " << m31() << ", " << m32() << ", " << m33() << ", "
-           << m34() << ", " << m41() << ", " << m42() << ", " << m43() << ", "
-           << m44();
+    result.append("matrix3d(");
+    result.append(String::NumberToStringECMAScript(m11()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m12()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m13()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m14()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m21()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m22()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m23()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m24()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m31()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m32()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m33()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m34()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m41()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m42()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m43()));
+    result.append(", ");
+    result.append(String::NumberToStringECMAScript(m44()));
   }
-  stream << ")";
+  result.append(")");
 
-  return String(stream.str().c_str());
+  return result;
 }
 
 ScriptValue DOMMatrixReadOnly::toJSONForBinding(
     ScriptState* script_state) const {
   V8ObjectBuilder result(script_state);
   result.AddNumber("a", a());
   result.AddNumber("b", b());
   result.AddNumber("c", c());
   result.AddNumber("d", d());
   result.AddNumber("e", e());
@@ skipping 62 matching lines @@
 
   matrix_->MakeIdentity();
   operations.Apply(FloatSize(0, 0), *matrix_);
 
   is2d_ = !operations.Has3DOperation();
 
   return;
 }
 
 }  // namespace blink