Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Unified Diff: third_party/WebKit/Source/core/css/CSSMatrix.cpp

Issue 2846523002: Update the stringifier behavior for DOMMatrixReadOnly (Closed)
Patch Set: Update the stringifier behavior for DOMMatrixReadOnly Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/css/CSSMatrix.cpp
diff --git a/third_party/WebKit/Source/core/css/CSSMatrix.cpp b/third_party/WebKit/Source/core/css/CSSMatrix.cpp
index ed126c89de06755bcf30a248b3556af77cce3482..227272c8a45d4118d818d329cd8e6bf102d5e83e 100644
--- a/third_party/WebKit/Source/core/css/CSSMatrix.cpp
+++ b/third_party/WebKit/Source/core/css/CSSMatrix.cpp
@@ -193,12 +193,12 @@ String CSSMatrix::toString() const {
// FIXME - Need to ensure valid CSS floating point values
// (https://bugs.webkit.org/show_bug.cgi?id=20674)
if (matrix_->IsAffine())
- return String::Format("matrix(%f, %f, %f, %f, %f, %f)", matrix_->A(),
+ return String::Format("matrix(%g, %g, %g, %g, %g, %g)", matrix_->A(),
zino 2017/06/06 09:20:08 I mean that this change should not be contained in
matrix_->B(), matrix_->C(), matrix_->D(),
matrix_->E(), matrix_->F());
return String::Format(
- "matrix3d(%f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, "
- "%f)",
+ "matrix3d(%g, %g, %g, %g, %g, %g, %g, %g, %g, %g, %g, %g, %g, %g, %g, "
+ "%g)",
matrix_->M11(), matrix_->M12(), matrix_->M13(), matrix_->M14(),
matrix_->M21(), matrix_->M22(), matrix_->M23(), matrix_->M24(),
matrix_->M31(), matrix_->M32(), matrix_->M33(), matrix_->M34(),

Powered by Google App Engine
This is Rietveld 408576698