[GeometryInterface] Add fromFloat32Array & fromFloat64Array function

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/DOMMatrix.h"
 
 namespace blink {
 
 DOMMatrixReadOnly* DOMMatrixReadOnly::create(Vector<double> sequence, ExceptionState& exceptionState)
 {
     if (sequence.size() != 6 && sequence.size() != 16) {
         exceptionState.throwTypeError("An invalid number sequence is specified. The sequence must contain 6 elements for 2D matrix and 16 elements for 3D matrix.");
         return nullptr;
     }
-    return new DOMMatrixReadOnly(sequence);
+    return new DOMMatrixReadOnly(sequence, sequence.size());
 }
 
-DOMMatrixReadOnly::DOMMatrixReadOnly(Vector<double> sequence)
+template <typename T>
+DOMMatrixReadOnly::DOMMatrixReadOnly(T sequence, int size)
 {
-    if (sequence.size() == 6) {
+    if (size == 6) {
         m_matrix = TransformationMatrix::create(
             sequence[0], sequence[1], sequence[2], sequence[3],
             sequence[4], sequence[5]);
         m_is2D = true;
-    } else if (sequence.size() == 16) {
+    } else if (size == 16) {
         m_matrix = TransformationMatrix::create(
             sequence[0], sequence[1], sequence[2], sequence[3],
             sequence[4], sequence[5], sequence[6], sequence[7],
             sequence[8], sequence[9], sequence[10], sequence[11],
             sequence[12], sequence[13], sequence[14], sequence[15]);
         m_is2D = false;
     } else {
         NOTREACHED();
     }
 }
 
+DOMMatrixReadOnly* DOMMatrixReadOnly::fromFloat32Array(DOMFloat32Array* float32Array, ExceptionState& exceptionState)
+{
+    if (float32Array->length() != 6 && float32Array->length() != 16) {
+        exceptionState.throwTypeError("An invalid number sequence is specified. The sequence must contain 6 elements for 2D matrix and 16 elements for 3D matrix.");

[dominicc (has gone to gerrit), 2016/09/06 17:07:22]

and 16 elements -> or 16 elements

Saying 'and' here implies you need 16+6=22 elements.

2D matrix -> a 2D matrix; 3D matrix -> a 3D matrix

"is specified" is a bit awkward because it is passive voice. Maybe shorter is
better; just omit it the first sentence?

Below too.

[Hwanseung Lee, 2016/09/07 14:21:47]

Done.

+        return nullptr;
+    }
+    return new DOMMatrixReadOnly(float32Array->data(), float32Array->length());
+}
+
+
+DOMMatrixReadOnly* DOMMatrixReadOnly::fromFloat64Array(DOMFloat64Array* float64Array, ExceptionState& exceptionState)
+{
+    if (float64Array->length() != 6 && float64Array->length() != 16) {
+        exceptionState.throwTypeError("An invalid number sequence is specified. The sequence must contain 6 elements for 2D matrix and 16 elements for 3D matrix.");
+        return nullptr;
+    }
+    return new DOMMatrixReadOnly(float64Array->data(), float64Array->length());
+}
+
 DOMMatrixReadOnly::~DOMMatrixReadOnly()
 {
 }
 
 bool DOMMatrixReadOnly::is2D() const
 {
     return m_is2D;
 }
 
 bool DOMMatrixReadOnly::isIdentity() const
@@ skipping 99 matching lines @@
         << m21() << ", " << m22() << ", " << m23() << ", " << m24() << ", "
         << m31() << ", " << m32() << ", " << m33() << ", " << m34() << ", "
         << m41() << ", " << m42() << ", " << m43() << ", " << m44();
     }
     stream << ")";
 
     return String(stream.str().c_str());
 }
 
 } // namespace blink