CONTINUED Make FrameSelection to hold non-canonicalized positions

third_party/WebKit/Source/core/layout/compositing/CompositedLayerMapping.cpp

 /*
  * Copyright (C) 2009, 2010, 2011 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 828 matching lines @@
     Vector<PaintLayer*>& layersNeedingPaintInvalidation) {
   if (!squashingLayer)
     return;
 
   LayoutPoint compositingContainerOffsetFromParentGraphicsLayer =
       -graphicsLayerParentLocation;
   if (compositingContainer)
     compositingContainerOffsetFromParentGraphicsLayer +=
         compositingContainer->subpixelAccumulation();
 
-#if DCHECK_IS_ON()
-  const PaintLayer* commonTransformAncestor = nullptr;
-  if (compositingContainer && compositingContainer->transform())
-    commonTransformAncestor = compositingContainer;
-  else if (compositingContainer)
-    commonTransformAncestor = compositingContainer->transformAncestor();
-#endif
+  // TODO(trchen): We should enable this for below comment out |DCHECK()| once
+  // we have simple reproduce case and fix it. See http://crbug.com/646437 for
+  // details.
+  // const PaintLayer* commonTransformAncestor = nullptr;
+  // if (compositingContainer && compositingContainer->transform())
+  //   commonTransformAncestor = compositingContainer;
+  // else if (compositingContainer)
+  //   commonTransformAncestor = compositingContainer->transformAncestor();
+
   // FIXME: Cache these offsets.
   LayoutPoint compositingContainerOffsetFromTransformedAncestor;
   if (compositingContainer && !compositingContainer->transform())
     compositingContainerOffsetFromTransformedAncestor =
         compositingContainer->computeOffsetFromTransformedAncestor();
 
   LayoutRect totalSquashBounds;
   for (size_t i = 0; i < layers.size(); ++i) {
     LayoutRect squashedBounds =
         layers[i].paintLayer->boundingBoxForCompositing();
 
     // Store the local bounds of the Layer subtree before applying the offset.
     layers[i].compositedBounds = squashedBounds;
 
-#if DCHECK_IS_ON()
-    DCHECK(layers[i].paintLayer->transformAncestor() ==
-           commonTransformAncestor);
-#endif
+    // TODO(trchen): We should enable this |DCHECK()| once we have simple
+    // reproduce case and fix it. See http://crbug.com/646437 for details.
+    // DCHECK(layers[i].paintLayer->transformAncestor() ==
+    //        commonTransformAncestor);
     LayoutPoint squashedLayerOffsetFromTransformedAncestor =
         layers[i].paintLayer->computeOffsetFromTransformedAncestor();
     LayoutSize squashedLayerOffsetFromCompositingContainer =
         squashedLayerOffsetFromTransformedAncestor -
         compositingContainerOffsetFromTransformedAncestor;
 
     squashedBounds.move(squashedLayerOffsetFromCompositingContainer);
     totalSquashBounds.unite(squashedBounds);
   }
 
@@ skipping 2585 matching lines @@
   } else if (graphicsLayer == m_decorationOutlineLayer.get()) {
     name = "Decoration Layer";
   } else {
     ASSERT_NOT_REACHED();
   }
 
   return name;
 }
 
 }  // namespace blink