Rename some functions about layout locations

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2006, 2007 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 309 matching lines @@
   LayoutSize size() const { return m_frameRect.size(); }
   IntSize pixelSnappedSize() const { return m_frameRect.pixelSnappedSize(); }
 
   void setLocation(const LayoutPoint& location) {
     if (location == m_frameRect.location())
       return;
     m_frameRect.setLocation(location);
     frameRectChanged();
   }
 
-  // The ancestor box that this object's location and topLeftLocation are
+  // The ancestor box that this object's location and physicalLocation are
   // relative to.
   virtual LayoutBox* locationContainer() const;
 
   // FIXME: Currently scrollbars are using int geometry and positioned based on
   // pixelSnappedBorderBoxRect whose size may change when location changes
   // because of pixel snapping. This function is used to change location of the
   // LayoutBox outside of LayoutBox::layout(). Will remove when we use
   // LayoutUnits for scrollbars.
   void setLocationAndUpdateOverflowControlsIfNeeded(const LayoutPoint&);
 
@@ skipping 831 matching lines @@
   flipForWritingMode(const FloatPoint& position) const {
     if (!UNLIKELY(hasFlippedBlocksWritingMode()))
       return position;
     return FloatPoint(m_frameRect.width() - position.x(), position.y());
   }
   void flipForWritingMode(FloatRect& rect) const {
     if (!UNLIKELY(hasFlippedBlocksWritingMode()))
       return;
     rect.setX(m_frameRect.width() - rect.maxX());
   }
-  // These represent your location relative to your container as a physical
-  // offset. In layout related methods you almost always want the logical
-  // location (e.g. x() and y()). Passing |container| causes flipped-block
-  // flipping w.r.t. that container, or containingBlock() otherwise.
-  LayoutPoint topLeftLocation(
+
+  // Passing |container| causes flipped-block flipping w.r.t. that container,
+  // or containingBlock() otherwise.
+  LayoutPoint physicalLocation(
       const LayoutBox* flippedBlocksContainer = nullptr) const;
-  LayoutSize topLeftLocationOffset() const {
-    return toLayoutSize(topLeftLocation());
+  LayoutSize physicalLocationOffset() const {
+    return toLayoutSize(physicalLocation());
   }
 
   LayoutRect logicalVisualOverflowRectForPropagation(
       const ComputedStyle&) const;
   LayoutRect visualOverflowRectForPropagation(const ComputedStyle&) const;
   LayoutRect logicalLayoutOverflowRectForPropagation(
       const ComputedStyle&) const;
   LayoutRect layoutOverflowRectForPropagation(const ComputedStyle&) const;
 
   bool hasOverflowModel() const { return m_overflow.get(); }
@@ skipping 423 matching lines @@
 
 inline bool LayoutBox::isForcedFragmentainerBreakValue(EBreak breakValue) {
   return breakValue == BreakColumn || breakValue == BreakLeft ||
          breakValue == BreakPage || breakValue == BreakRecto ||
          breakValue == BreakRight || breakValue == BreakVerso;
 }
 
 }  // namespace blink
 
 #endif  // LayoutBox_h