Implement collection of out-of-flow descendants

third_party/WebKit/Source/core/layout/ng/ng_units.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NGUnits_h
 #define NGUnits_h
 
 #include "core/CoreExport.h"
 #include "core/layout/ng/ng_writing_mode.h"
 #include "platform/LayoutUnit.h"
@@ skipping 39 matching lines @@
 struct NGLogicalOffset {
   NGLogicalOffset() {}
   NGLogicalOffset(LayoutUnit inline_offset, LayoutUnit block_offset)
       : inline_offset(inline_offset), block_offset(block_offset) {}
 
   LayoutUnit inline_offset;
   LayoutUnit block_offset;
 
   // Converts a logical offset to a physical offset. See:
   // https://drafts.csswg.org/css-writing-modes-3/#logical-to-physical
+  // PhysicalOffset will be the physical top left point of the rectangle
+  // described by offset + inner_size. Setting inner_size to 0,0 will return
+  // the same point.
   // @param outer_size the size of the rect (typically a fragment).
   // @param inner_size the size of the inner rect (typically a child fragment).
   CORE_EXPORT NGPhysicalOffset
   ConvertToPhysical(NGWritingMode,
                     TextDirection,
                     NGPhysicalSize outer_size,
                     NGPhysicalSize inner_size) const;
+
   bool operator==(const NGLogicalOffset& other) const;
 
   NGLogicalOffset operator+(const NGLogicalOffset& other) const;
 
   NGLogicalOffset& operator+=(const NGLogicalOffset& other);
 
   bool operator>(const NGLogicalOffset& other) const;
   bool operator>=(const NGLogicalOffset& other) const;
 
   bool operator<(const NGLogicalOffset& other) const;
   bool operator<=(const NGLogicalOffset& other) const;
 
   String ToString() const;
 };
 
 CORE_EXPORT inline std::ostream& operator<<(std::ostream& os,
                                             const NGLogicalOffset& value) {
   return os << value.ToString();
 }
 
 // NGPhysicalOffset is the position of a rect (typically a fragment) relative to
 // its parent rect in the physical coordinate system.
 struct NGPhysicalOffset {
   NGPhysicalOffset() {}
   NGPhysicalOffset(LayoutUnit left, LayoutUnit top) : left(left), top(top) {}
 
   LayoutUnit left;
   LayoutUnit top;
+
+  NGPhysicalOffset operator+(const NGPhysicalOffset& other) const {

[ikilpatrick, 2016/12/02 17:47:06]

also define:
NGPhysicalOffset& operator+=(const NGPhysicalOffset other) const;

also probably move to .cc file to be consistent with NGLogicalOffset.

[atotic, 2016/12/02 19:55:16]

done. 

+= is not used yet, our current policy seems to be "define some operators, but
not other". Also, some are defined inline (NGBoxStrut), some not.

[ikilpatrick, 2016/12/02 21:06:13]

Thanks - this comes from:
https://google.github.io/styleguide/cppguide.html#Operator_Overloading
"If you define an operator, also define any related operators that make sense,
and make sure they are defined consistently."

+    return NGPhysicalOffset{this->left + other.left, this->top + other.top};
+  }
 };
 
 struct NGPhysicalSize {
   NGPhysicalSize() {}
   NGPhysicalSize(LayoutUnit width, LayoutUnit height)
       : width(width), height(height) {}
 
   LayoutUnit width;
   LayoutUnit height;
 
@@ skipping 155 matching lines @@
                                 const NGMarginStrut& value) {
   return stream << value.ToString();
 }
 
 // Struct to represent a simple edge that has start and end.
 struct NGEdge {
   LayoutUnit start;
   LayoutUnit end;
 };
 
+// Represents rectangle's physical corner.

[ikilpatrick, 2016/12/02 17:47:06]

Maybe:

This struct is used to represent the static position of an out of flow
descendant.

(similar to NGMarginStruct, specifies which part of layout it's used for).

[atotic, 2016/12/02 19:55:16]

done

+struct CORE_EXPORT NGCorner {
+  enum Type { kTopLeft, kTopRight, kBottomLeft, kBottomRight };
+
+  Type type;  // Logical corner that corresponds to physical top left.

[ikilpatrick, 2016/12/02 17:47:06]

remove this comment?

[atotic, 2016/12/02 19:55:16]

Not done. I will not know what type is a year from now if this comment is
removed.

+  NGPhysicalOffset offset;
+
+  // Creates a corner with proper type wrt writing mode and direction.
+  static NGCorner Create(NGWritingMode, TextDirection, NGPhysicalOffset);
+};
+
 }  // namespace blink
 
 #endif  // NGUnits_h