Use NGLogicalRect instead of NGExclusion for exclusions.

third_party/WebKit/Source/core/layout/ng/ng_fragment.cc

Index: third_party/WebKit/Source/core/layout/ng/ng_fragment.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_fragment.cc b/third_party/WebKit/Source/core/layout/ng/ng_fragment.cc
index ca9dcb63a722bd2fbd1c173c590286b3d543cbe3..6c5cdbd5b51040294b9c17d231aab4a1daf5d3d7 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_fragment.cc
+++ b/third_party/WebKit/Source/core/layout/ng/ng_fragment.cc
@@ -4,14 +4,15 @@
 
 #include "core/layout/ng/ng_fragment.h"
 
+#include "core/layout/ng/ng_macros.h"
 #include "core/layout/ng/ng_physical_fragment.h"
 
 namespace blink {
 
 NGMarginStrut NGFragment::MarginStrut() const {
-  // NOTE: Accessing the margin strut ignoring the writing mode here is fine.
-  // Changing the writing mode establishes a new formatting context, for which

[cbiesinger, 2016/11/01 15:02:01]

You deleted this line, and now the description doesn't make sense anymore...

-  // a margin strut is never set for a fragment.
+  WRITING_MODE_IGNORED(
+      "Accessing the margin strut ignoring the writing mode here is fine."
+      "A margin strut is never set for a fragment");
   return toNGPhysicalFragment(physical_fragment_)->MarginStrut();
 }