Make WebKitCSSMatrix an alias of DOMMatrix

third_party/WebKit/Source/core/dom/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/dom/DOMMatrixReadOnly.h"
 
 #include "bindings/core/v8/V8ObjectBuilder.h"
 #include "core/css/CSSIdentifierValue.h"
 #include "core/css/CSSToLengthConversionData.h"
 #include "core/css/CSSValueList.h"
@@ skipping 282 matching lines @@
   double array[] = {
       m_matrix->m11(), m_matrix->m12(), m_matrix->m13(), m_matrix->m14(),
       m_matrix->m21(), m_matrix->m22(), m_matrix->m23(), m_matrix->m24(),
       m_matrix->m31(), m_matrix->m32(), m_matrix->m33(), m_matrix->m34(),
       m_matrix->m41(), m_matrix->m42(), m_matrix->m43(), m_matrix->m44()};
 
   return DOMFloat64Array::create(array, 16);
 }
 
 const String DOMMatrixReadOnly::toString() const {
-  std::stringstream stream;
   if (is2D()) {
-    stream << "matrix(" << a() << ", " << b() << ", " << c() << ", " << d()
-           << ", " << e() << ", " << f();
+    return String::format("matrix(%f, %f, %f, %f, %f, %f)", a(), b(), c(), d(),

[foolip, 2017/03/21 08:47:30]

Is this a drive-by change, or does it affect the output in  any case?

[Byoungkwon Ko, 2017/04/26 01:31:35]

yes, when we were using steam, we should fix decimal fraction part for float
variable.
So I changed to string::format.
Is it better to use precision() ???

[foolip, 2017/04/26 07:29:19]

Hmm, this is actually related to https://github.com/w3c/fxtf-drafts/issues/120.
Should double precision be used here?

In any case, I would suggest doing this change separately.

+                          e(), f());
   } else {
-    stream << "matrix3d(" << m11() << ", " << m12() << ", " << m13() << ", "
-           << m14() << ", " << m21() << ", " << m22() << ", " << m23() << ", "
-           << m24() << ", " << m31() << ", " << m32() << ", " << m33() << ", "
-           << m34() << ", " << m41() << ", " << m42() << ", " << m43() << ", "
-           << m44();
+    return String::format(
+        "matrix3d(%f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, "
+        "%f)",
+        m11(), m12(), m13(), m14(), m21(), m22(), m23(), m24(), m31(), m32(),
+        m33(), m34(), m41(), m42(), m43(), m44());
   }
-  stream << ")";
-
-  return String(stream.str().c_str());
 }
 
 ScriptValue DOMMatrixReadOnly::toJSONForBinding(
     ScriptState* scriptState) const {
   V8ObjectBuilder result(scriptState);
   result.addNumber("a", a());
   result.addNumber("b", b());
   result.addNumber("c", c());
   result.addNumber("d", d());
   result.addNumber("e", e());
@@ skipping 62 matching lines @@
 
   m_matrix->makeIdentity();
   operations.apply(FloatSize(0, 0), *m_matrix);
 
   m_is2D = !operations.has3DOperation();
 
   return;
 }
 
 }  // namespace blink