[LayoutNG] Switch NGBreakToken to being RefCounted.

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

Index: third_party/WebKit/Source/core/layout/ng/ng_block_child_iterator.h
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_block_child_iterator.h b/third_party/WebKit/Source/core/layout/ng/ng_block_child_iterator.h
index 0aee68ee038e46ac303fdcd0893cc926f2d92b1f..e8e9becaafce694db3e7a72657210859d30bc257 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_block_child_iterator.h
+++ b/third_party/WebKit/Source/core/layout/ng/ng_block_child_iterator.h
@@ -7,6 +7,7 @@
 
 #include "core/CoreExport.h"
 #include "platform/heap/Handle.h"
+#include "wtf/RefPtr.h"
 
 namespace blink {
 
@@ -20,6 +21,8 @@ class NGBlockBreakToken;
 // This class does not handle modifications to its arguments after it has been
 // constructed.
 class CORE_EXPORT NGBlockChildIterator {
+  STACK_ALLOCATED();
+
  public:
   NGBlockChildIterator(NGLayoutInputNode* first_child,
                        NGBlockBreakToken* break_token);
@@ -31,7 +34,7 @@ class CORE_EXPORT NGBlockChildIterator {
 
  private:
   Persistent<NGLayoutInputNode> child_;
-  Persistent<NGBlockBreakToken> break_token_;
+  NGBlockBreakToken* break_token_;  // Not-owned as STACK_ALLOCATED.

[ikilpatrick, 2017/02/28 17:13:13]

it'd be nice if we had a class like:
NotARefPtrBecuaseStackAllocated<>

 
   // An index into break_token_'s ChildBreakTokens() vector. Used for keeping
   // track of the next child break token to inspect.
@@ -39,11 +42,13 @@ class CORE_EXPORT NGBlockChildIterator {
 };
 
 struct NGBlockChildIterator::Entry {
+  STACK_ALLOCATED();
+
   Entry(NGLayoutInputNode* node, NGBreakToken* token)
       : node(node), token(token) {}
 
   Persistent<NGLayoutInputNode> node;
-  Persistent<NGBreakToken> token;
+  NGBreakToken* token;  // Not-owned as STACK_ALLOCATED.
 
   bool operator==(const NGBlockChildIterator::Entry& other) const {
     return node == other.node && token == other.token;