Use Opportunity Iterator to position text fragments in NGLineBuilder

third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.cc

Index: third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.cc b/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.cc
index 5c7edf0b0ee5773766e7ec303d8da5aeeb9fe472..e5b47d1e1f774f0c9a55a7dd7ce1bd5890741165 100644
--- a/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.cc
+++ b/third_party/WebKit/Source/core/layout/ng/geometry/ng_logical_offset.cc
@@ -56,6 +56,11 @@ bool NGLogicalOffset::operator==(const NGLogicalOffset& other) const {
          std::tie(inline_offset, block_offset);
 }
 
+bool NGLogicalOffset::operator!=(const NGLogicalOffset& other) const {
+  return std::tie(other.inline_offset, other.block_offset) !=

[ikilpatrick, 2017/03/09 22:00:51]

just "return !operator==(other);" ?

[Gleb Lanbin, 2017/03/09 22:26:44]

nice, thanks!

+         std::tie(inline_offset, block_offset);
+}
+
 NGLogicalOffset NGLogicalOffset::operator+(const NGLogicalOffset& other) const {
   NGLogicalOffset result;
   result.inline_offset = this->inline_offset + other.inline_offset;