Fix the assert and clamping in SkMatrix::get*Scale[s]().

src/core/SkMatrix.cpp

Index: src/core/SkMatrix.cpp
diff --git a/src/core/SkMatrix.cpp b/src/core/SkMatrix.cpp
index fc698530ec9d2b01a31778d70a5198b5ea61e993..0fd802087fc0c5f52e6da2be56edbfb1b9fcc442 100644
--- a/src/core/SkMatrix.cpp
+++ b/src/core/SkMatrix.cpp
@@ -1571,19 +1571,22 @@ template <MinMaxOrBoth MIN_MAX_OR_BOTH> bool get_scale_factor(SkMatrix::TypeMask
     if (!SkScalarIsFinite(results[0])) {
         return false;
     }
-    if (results[0] < 0 && results[0] > -SK_ScalarNearlyZero) {
+    // Due to the floating point inaccuracy, there might be an error in a, b, c
+    // calculated by sdot, further deepened by subsequent arithmetic operations
+    // on them. Therefore, we allow and cap the nearly-zero negative values.
+    SkASSERT(results[0] >= -SK_ScalarNearlyZero);
+    if (results[0] < 0) {
         results[0] = 0;
     }
-    SkASSERT(results[0] >= 0);
     results[0] = SkScalarSqrt(results[0]);
     if (kBoth_MinMaxOrBoth == MIN_MAX_OR_BOTH) {
         if (!SkScalarIsFinite(results[1])) {
             return false;
         }
-        if (results[1] < 0 && results[1] > -SK_ScalarNearlyZero) {
+        SkASSERT(results[1] >= -SK_ScalarNearlyZero);
+        if (results[1] < 0) {
             results[1] = 0;
         }
-        SkASSERT(results[1] >= 0);
         results[1] = SkScalarSqrt(results[1]);
     }
     return true;