Out-of-line trace() methods of editing template types.

third_party/WebKit/Source/core/editing/EphemeralRange.h

 // Copyright 2015 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 EphemeralRange_h
 #define EphemeralRange_h
 
 #include "core/editing/Position.h"
 
 namespace blink {
@@ skipping 97 matching lines @@
   // Returns true if |m_startPositoin| == |m_endPosition| or |isNull()|.
   bool isCollapsed() const;
   bool isNull() const {
     DCHECK(isValid());
     return m_startPosition.isNull();
   }
   bool isNotNull() const { return !isNull(); }
 
   RangeTraversal nodes() const;
 
-  DEFINE_INLINE_TRACE() {

[sof, 2017/02/07 11:05:57]

This was an unused trace method (the class is STACK_ALLOCATED()), which I notice
the GC plugin doesn't check for. Would it be worth adding a warning for this, or
should we just silently allow it?

-    visitor->trace(m_startPosition);
-    visitor->trace(m_endPosition);
-  }
-
   // |node| should be in-document and valid for anchor node of
   // |PositionTemplate<Strategy>|.
   static EphemeralRangeTemplate<Strategy> rangeOfContents(
       const Node& /* node */);
 
  private:
   bool isValid() const;
 
   PositionTemplate<Strategy> m_startPosition;
   PositionTemplate<Strategy> m_endPosition;
@@ skipping 11 matching lines @@
 using EphemeralRangeInFlatTree =
     EphemeralRangeTemplate<EditingInFlatTreeStrategy>;
 
 // Returns a newly created |Range| object from |range| or |nullptr| if
 // |range.isNull()| returns true.
 CORE_EXPORT Range* createRange(const EphemeralRange& /* range */);
 
 }  // namespace blink
 
 #endif