Fix matrix3d transform under page zoom

third_party/WebKit/Source/core/css/resolver/TransformBuilder.cpp

Index: third_party/WebKit/Source/core/css/resolver/TransformBuilder.cpp
diff --git a/third_party/WebKit/Source/core/css/resolver/TransformBuilder.cpp b/third_party/WebKit/Source/core/css/resolver/TransformBuilder.cpp
index 0c00516edf56a13f409d28cad8b80c05c6cab350..4d6c0385f4cb546ca387267caf4ae9b17d0ee1da 100644
--- a/third_party/WebKit/Source/core/css/resolver/TransformBuilder.cpp
+++ b/third_party/WebKit/Source/core/css/resolver/TransformBuilder.cpp
@@ -296,20 +296,24 @@ TransformOperations TransformBuilder::createTransformOperations(
             toCSSPrimitiveValue(transformValue->item(0)).getDoubleValue(),
             toCSSPrimitiveValue(transformValue->item(1)).getDoubleValue(),
             toCSSPrimitiveValue(transformValue->item(2)).getDoubleValue(),
-            toCSSPrimitiveValue(transformValue->item(3)).getDoubleValue(),
+            toCSSPrimitiveValue(transformValue->item(3)).getDoubleValue() /
+                zoomFactor,
             toCSSPrimitiveValue(transformValue->item(4)).getDoubleValue(),
             toCSSPrimitiveValue(transformValue->item(5)).getDoubleValue(),
             toCSSPrimitiveValue(transformValue->item(6)).getDoubleValue(),
-            toCSSPrimitiveValue(transformValue->item(7)).getDoubleValue(),
+            toCSSPrimitiveValue(transformValue->item(7)).getDoubleValue() /
+                zoomFactor,
             toCSSPrimitiveValue(transformValue->item(8)).getDoubleValue(),
             toCSSPrimitiveValue(transformValue->item(9)).getDoubleValue(),
             toCSSPrimitiveValue(transformValue->item(10)).getDoubleValue(),
-            toCSSPrimitiveValue(transformValue->item(11)).getDoubleValue(),
+            toCSSPrimitiveValue(transformValue->item(11)).getDoubleValue() /
+                zoomFactor,
             zoomFactor *
                 toCSSPrimitiveValue(transformValue->item(12)).getDoubleValue(),
             zoomFactor *
                 toCSSPrimitiveValue(transformValue->item(13)).getDoubleValue(),
-            toCSSPrimitiveValue(transformValue->item(14)).getDoubleValue(),
+            zoomFactor *

[pdr., 2016/11/08 07:04:08]

WDYT of extracting this out of the loop and only applying it at the end?

Something like:
for (auto& value : toCSSValueList(inValue)) {
   ... (case stmts)
}

float zoomFactor = conversionData.zoom();
if (zoomFactor != 1)
    operations = operations.zoom(zoomFactor);
return operations;

[Franklin Ta, 2016/11/08 07:52:18]

You also have to refactor the zoom adjustment in the other cases(perspective and
translate) if you want to move all of that logic into operations.zoom so it
might end up being a messier change. For example in the perspective case I think
I might need to unzoom the value from computeLength just so it can be zoomed
again later.

Can I leave this for someone a bit more experienced with the code base? I was
hoping this CL can just be for correctness and alancutter can do the refactor
(or assign it to someone) since he added the zoom method in operations.

[Franklin Ta, 2016/11/08 16:47:02]

Btw @alancutter if you're not the owner for this, don't worry about it. I was
assuming that you dealt with the same problem elsewhere when you added that zoom
method so you would know how to best refactor this. I just took another look at
where TransformOperations::zoom is used inside AnimationStyleBuilder and it
doesn't look like it uses similar code for handling zoom at all.

The main complexity here is in the perspective and translate cases where you
need to calculate them using unzoomed lengths. The call hierarchy for
convertToFloatLength is pretty deep (to me) to figure out when exactly
zoomFactor got multiplied in to see how to remove it.

For example: it goes convertToFloatLength -> convertToLength -> computeLength ->
computeLengthDouble -> zoomedComputedPixels but not when it's
em/rem/percentage/calc/etc. I think it might just a multiply by the zoomFactor
at some point for all cases so you can just divide the existing code by the
zoomFactor to undo the zoom but this has to be verified and tested.

Anyway, if there's a refactor necessary here I might not be the best person to
help out with this. =(

+                toCSSPrimitiveValue(transformValue->item(14)).getDoubleValue(),
             toCSSPrimitiveValue(transformValue->item(15)).getDoubleValue());
         operations.operations().append(
             Matrix3DTransformOperation::create(matrix));