Account for perspective and preserve-3d in mapToVisualRectInAncestorSpace

third_party/WebKit/Source/core/layout/LayoutBox.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc.
  *               All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 1142 matching lines @@
   if (hasOverflowClip() || style()->containsPaint())
     result = overflowClipRect(LayoutPoint());
 
   if (hasClip())
     result.intersect(clipRect(LayoutPoint()));
 
   return result;
 }
 
 bool LayoutBox::mapScrollingContentsRectToBoxSpace(
-    LayoutRect& rect,
+    TransformState& transformState,
+    TransformState::TransformAccumulation accumulation,
     VisualRectFlags visualRectFlags) const {
   if (!hasClipRelatedProperty())
     return true;
 
   if (hasOverflowClip()) {
     LayoutSize offset = LayoutSize(-scrolledContentOffset());
-    rect.move(offset);
+    transformState.move(offset, accumulation);
   }
 
   // This won't work fully correctly for fixed-position elements, who should
   // receive CSS clip but for whom the current object is not in the containing
   // block chain.
   LayoutRect clipRect = clippingRect();
 
+  transformState.flatten();
+  LayoutRect rect(transformState.lastPlanarQuad().enclosingBoundingBox());
   bool doesIntersect;
   if (visualRectFlags & EdgeInclusive) {
     doesIntersect = rect.inclusiveIntersect(clipRect);
   } else {
     rect.intersect(clipRect);
     doesIntersect = !rect.isEmpty();
   }
+  transformState.setQuad(FloatQuad(FloatRect(rect)));
 
   return doesIntersect;
 }
 
 void LayoutBox::computeIntrinsicLogicalWidths(
     LayoutUnit& minLogicalWidth,
     LayoutUnit& maxLogicalWidth) const {
   minLogicalWidth = minPreferredLogicalWidth() - borderAndPaddingLogicalWidth();
   maxLogicalWidth = maxPreferredLogicalWidth() - borderAndPaddingLogicalWidth();
 }
@@ skipping 1126 matching lines @@
 LayoutRect LayoutBox::localVisualRect() const {
   if (style()->visibility() != EVisibility::kVisible)
     return LayoutRect();
 
   if (hasMask() && !RuntimeEnabledFeatures::slimmingPaintV2Enabled())
     return LayoutRect(layer()->boxForFilterOrMask());
   return selfVisualOverflowRect();
 }
 
 void LayoutBox::inflateVisualRectForFilterUnderContainer(
-    LayoutRect& rect,
+    TransformState& transformState,
     const LayoutObject& container,
     const LayoutBoxModelObject* ancestorToStopAt) const {
+  transformState.flatten();
   // Apply visual overflow caused by reflections and filters defined on objects
   // between this object and container (not included) or ancestorToStopAt
   // (included).
   LayoutSize offsetFromContainer = this->offsetFromContainer(&container);
-  rect.move(offsetFromContainer);
+  transformState.move(offsetFromContainer);
   for (LayoutObject* parent = this->parent(); parent && parent != container;
        parent = parent->parent()) {
     if (parent->isBox()) {
       // Convert rect into coordinate space of parent to apply parent's
       // reflection and filter.
       LayoutSize parentOffset = parent->offsetFromAncestorContainer(&container);
-      rect.move(-parentOffset);
-      toLayoutBox(parent)->inflateVisualRectForFilter(rect);
-      rect.move(parentOffset);
+      transformState.move(-parentOffset);
+      toLayoutBox(parent)->inflateVisualRectForFilter(transformState);
+      transformState.move(parentOffset);
     }
     if (parent == ancestorToStopAt)
       break;
   }
-  rect.move(-offsetFromContainer);
+  transformState.move(-offsetFromContainer);
 }
 
-bool LayoutBox::mapToVisualRectInAncestorSpace(
+bool LayoutBox::mapToVisualRectInAncestorSpaceInternal(
     const LayoutBoxModelObject* ancestor,
-    LayoutRect& rect,
+    TransformState& transformState,
     VisualRectFlags visualRectFlags) const {
-  inflateVisualRectForFilter(rect);
+  inflateVisualRectForFilter(transformState);
 
   if (ancestor == this)
     return true;
 
   AncestorSkipInfo skipInfo(ancestor, true);
   LayoutObject* container = this->container(&skipInfo);
   LayoutBox* tableRowContainer = nullptr;
   // Skip table row because cells and rows are in the same coordinate space (see
   // below, however for more comments about when |ancestor| is the table row).
   if (isTableCell()) {
     DCHECK(container->isTableRow() && parentBox() == container);
     if (container != ancestor)
       container = container->parent();
     else
       tableRowContainer = toLayoutBox(container);
   }
   if (!container)
     return true;
 
-  if (skipInfo.filterSkipped())
-    inflateVisualRectForFilterUnderContainer(rect, *container, ancestor);
+  LayoutPoint containerOffset;
+  if (container->isBox()) {
+    containerOffset.moveBy(physicalLocation(toLayoutBox(container)));
 
-  // We are now in our parent container's coordinate space. Apply our transform
-  // to obtain a bounding box in the parent's coordinate space that encloses us.
-  if (hasLayer() && layer()->transform()) {
-    // Use enclosingIntRect because we cannot properly compute pixel snapping
-    // for painted elements within the transform since we don't know the desired
-    // subpixel accumulation at this point, and the transform may include a
-    // scale.
-    rect = LayoutRect(layer()->transform()->mapRect(enclosingIntRect(rect)));
-  }
-  LayoutPoint topLeft = rect.location();
-  if (container->isBox()) {
-    topLeft.moveBy(physicalLocation(toLayoutBox(container)));
     // If the row is the ancestor, however, add its offset back in. In effect,
     // this passes from the joint <td> / <tr> coordinate space to the parent
     // space, then back to <tr> / <td>.
     if (tableRowContainer) {
-      topLeft.moveBy(
+      containerOffset.moveBy(
           -tableRowContainer->physicalLocation(toLayoutBox(container)));
     }
   } else if (container->isRuby()) {
     // TODO(wkorman): Generalize Ruby specialization and/or document more
     // clearly. See the accompanying specialization in
-    // LayoutInline::mapToVisualRectInAncestorSpace.
-    topLeft.moveBy(physicalLocation());
+    // LayoutInline::mapToVisualRectInAncestorSpaceInternal.
+    containerOffset.moveBy(physicalLocation());
   } else {
-    topLeft.moveBy(location());
+    containerOffset.moveBy(location());
   }
 
   const ComputedStyle& styleToUse = styleRef();
   EPosition position = styleToUse.position();
   if (position == EPosition::kAbsolute && container->isInFlowPositioned() &&
       container->isLayoutInline()) {
-    topLeft +=
-        toLayoutInline(container)->offsetForInFlowPositionedInline(*this);
+    containerOffset.move(
+        toLayoutInline(container)->offsetForInFlowPositionedInline(*this));
   } else if (styleToUse.hasInFlowPosition() && layer()) {
     // Apply the relative position offset when invalidating a rectangle. The
     // layer is translated, but the layout box isn't, so we need to do this to
     // get the right dirty rect.  Since this is called from
     // LayoutObject::setStyle, the relative position flag on the LayoutObject
     // has been cleared, so use the one on the style().
-    topLeft += layer()->offsetForInFlowPosition();
+    containerOffset.move(layer()->offsetForInFlowPosition());
+  }
+
+  bool preserve3D =
+      ((container->style()->preserves3D() && !container->isText()) ||
+       (style()->preserves3D() && !isText()));

[Xianzhu, 2017/03/03 01:24:42]

I think !container->isText() and !isText() are both always true so can be
omitted.

[chrishtr, 2017/03/03 02:45:11]

Done.

+
+  TransformState::TransformAccumulation accumulation =
+      preserve3D ? TransformState::AccumulateTransform
+                 : TransformState::FlattenTransform;
+
+  if (skipInfo.filterSkipped()) {
+    inflateVisualRectForFilterUnderContainer(transformState, *container,
+                                             ancestor);
+  }
+
+  // We are now in our parent container's coordinate space. Apply our transform
+  // to obtain a bounding box in the parent's coordinate space that encloses us.
+  if (shouldUseTransformFromContainer(container)) {
+    TransformationMatrix t;
+    getTransformFromContainer(container, toLayoutSize(containerOffset), t);
+    transformState.applyTransform(t, accumulation);
+
+    // Use enclosingIntRect because we cannot properly compute pixel snapping

[Xianzhu, 2017/03/03 01:24:42]

enclosingIntRect -> enclosingBoundingBox().

Not a problem of this CL, but the name of FloatQuad::enclosingBoundingBox()
doesn't look good because the enclosingIntRect() semantics is not obvious from
the name. This can be addressed with a follow-up.

[chrishtr, 2017/03/03 02:45:11]

Will fix in a followup. Changed comment.

+    // for painted elements within the transform since we don't know the desired
+    // subpixel accumulation at this point, and the transform may include a
+    // scale.
+    if (!preserve3D) {
+      transformState.flatten();
+      transformState.setQuad(
+          FloatQuad(transformState.lastPlanarQuad().enclosingBoundingBox()));
+    }
+  } else {
+    transformState.move(toLayoutSize(containerOffset), accumulation);
   }
 
   // FIXME: We ignore the lightweight clipping rect that controls use, since if
   // |o| is in mid-layout, its controlClipRect will be wrong. For overflow clip
   // we use the values cached by the layer.
-  rect.setLocation(topLeft);
-
   if (container->isBox() && container != ancestor &&
       !toLayoutBox(container)->mapScrollingContentsRectToBoxSpace(
-          rect, visualRectFlags))
+          transformState, accumulation, visualRectFlags))
     return false;
 
   if (skipInfo.ancestorSkipped()) {
     // If the ancestor is below the container, then we need to map the rect into
     // ancestor's coordinates.
     LayoutSize containerOffset =
         ancestor->offsetFromAncestorContainer(container);
-    rect.move(-containerOffset);
+    transformState.move(-containerOffset, accumulation);
     // If the ancestor is fixed, then the rect is already in its coordinates so
     // doesn't need viewport-adjusting.
     if (ancestor->style()->position() != EPosition::kFixed &&
-        container->isLayoutView() && position == EPosition::kFixed)
-      rect.move(toLayoutView(container)->offsetForFixedPosition(true));
+        container->isLayoutView() && position == EPosition::kFixed) {
+      transformState.move(toLayoutView(container)->offsetForFixedPosition(true),
+                          accumulation);
+    }
     return true;
   }
 
   if (container->isLayoutView())
-    return toLayoutView(container)->mapToVisualRectInAncestorSpace(
-        ancestor, rect, position == EPosition::kFixed ? IsFixed : 0,
+    return toLayoutView(container)->mapToVisualRectInAncestorSpaceInternal(
+        ancestor, transformState, position == EPosition::kFixed ? IsFixed : 0,
         visualRectFlags);
   else
-    return container->mapToVisualRectInAncestorSpace(ancestor, rect,
-                                                     visualRectFlags);
+    return container->mapToVisualRectInAncestorSpaceInternal(
+        ancestor, transformState, visualRectFlags);
 }
 
-void LayoutBox::inflateVisualRectForFilter(LayoutRect& visualRect) const {
-  if (layer() && layer()->hasFilterInducingProperty())
-    visualRect = layer()->mapLayoutRectForFilter(visualRect);
+void LayoutBox::inflateVisualRectForFilter(
+    TransformState& transformState) const {
+  if (!layer() || !layer()->hasFilterInducingProperty())
+    return;
+
+  transformState.flatten();
+  LayoutRect rect(transformState.lastPlanarQuad().boundingBox());
+  transformState.setQuad(
+      FloatQuad(FloatRect(layer()->mapLayoutRectForFilter(rect))));
 }
 
 void LayoutBox::updateLogicalWidth() {
   LogicalExtentComputedValues computedValues;
   computeLogicalWidth(computedValues);
 
   setLogicalWidth(computedValues.m_extent);
   setLogicalLeft(computedValues.m_position);
   setMarginStart(computedValues.m_margins.m_start);
   setMarginEnd(computedValues.m_margins.m_end);
@@ skipping 3263 matching lines @@
 
 void LayoutBox::MutableForPainting::
     savePreviousContentBoxSizeAndLayoutOverflowRect() {
   auto& rareData = layoutBox().ensureRareData();
   rareData.m_hasPreviousContentBoxSizeAndLayoutOverflowRect = true;
   rareData.m_previousContentBoxSize = layoutBox().contentBoxRect().size();
   rareData.m_previousLayoutOverflowRect = layoutBox().layoutOverflowRect();
 }
 
 }  // namespace blink