blink style: Update one class cluster, delete one class, change comment style.

third_party/WebKit/Source/platform/heap/WrapperVisitor.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 WrapperVisitor_h
 #define WrapperVisitor_h
 
 #include "platform/PlatformExport.h"
 #include "platform/wtf/Allocator.h"
 
 namespace v8 {
 class Value;
 template <class T>
 class PersistentBase;
 }
 
 namespace blink {
 
 template <typename T>
 class TraceTrait;
 template <typename T>
 class Member;
 class ScriptWrappable;
 template <typename T>
 class TraceWrapperV8Reference;
 class TraceWrapperBase;
 template <typename T>
 class TraceWrapperMember;
 
-/**
- * Declares non-virtual traceWrappers method. Should be used on
- * non-ScriptWrappable classes which should participate in wrapper tracing (e.g.
- * StyleEngine):
- *
- *     class StyleEngine: public TraceWrapperBase {
- *      public:
- *       DECLARE_TRACE_WRAPPERS();
- *     };
- */
+// Declares non-virtual traceWrappers method. Should be used on
+// non-ScriptWrappable classes which should participate in wrapper tracing (e.g.
+// StyleEngine):
+//
+//     class StyleEngine: public TraceWrapperBase {
+//      public:
+//       DECLARE_TRACE_WRAPPERS();
+//     };
+//
 #define DECLARE_TRACE_WRAPPERS() \
   void TraceWrappers(const WrapperVisitor* visitor) const
 
-/**
- * Declares virtual traceWrappers method. It is used in ScriptWrappable, can be
- * used to override the method in the subclasses, and can be used by
- * non-ScriptWrappable classes which expect to be inherited.
- */
+// Declares virtual traceWrappers method. It is used in ScriptWrappable, can be
+// used to override the method in the subclasses, and can be used by
+// non-ScriptWrappable classes which expect to be inherited.
 #define DECLARE_VIRTUAL_TRACE_WRAPPERS() virtual DECLARE_TRACE_WRAPPERS()
 
-/**
- * Provides definition of traceWrappers method. Custom code will usually call
- * visitor->traceWrappers with all objects which could contribute to the set of
- * reachable wrappers:
- *
- *     DEFINE_TRACE_WRAPPERS(NodeRareData)
- *     {
- *         visitor->traceWrappers(m_nodeLists);
- *         visitor->traceWrappers(m_mutationObserverData);
- *     }
- */
+// Provides definition of traceWrappers method. Custom code will usually call
+// visitor->traceWrappers with all objects which could contribute to the set of
+// reachable wrappers:
+//
+//     DEFINE_TRACE_WRAPPERS(NodeRareData)
+//     {
+//         visitor->traceWrappers(node_lists_);

[dcheng, 2017/05/10 17:37:06]

Just to confirm: the comment CLs are mostly scoped at fixing m_foo and not so
much at functionCall(), right?

[Nico, 2017/05/10 17:42:43]

Yes, but done too, thanks. (I'm only focusing on things not-in-comments too, but
the c-style comment here defeated my cs query.)

+//         visitor->traceWrappers(mutation_observer_data_);
+//     }
+//
 #define DEFINE_TRACE_WRAPPERS(T) \
   void T::TraceWrappers(const WrapperVisitor* visitor) const
 
 #define DECLARE_TRACE_WRAPPERS_AFTER_DISPATCH() \
   void TraceWrappersAfterDispatch(const WrapperVisitor*) const
 
 #define DEFINE_TRACE_WRAPPERS_AFTER_DISPATCH(T) \
   void T::TraceWrappersAfterDispatch(const WrapperVisitor* visitor) const
 
 #define DEFINE_INLINE_TRACE_WRAPPERS() DECLARE_TRACE_WRAPPERS()
@@ skipping 28 matching lines @@
     }
 
     if (TraceTrait<T>::GetHeapObjectHeader(traceable)
             ->IsWrapperHeaderMarked()) {
       return;
     }
 
     MarkAndPushToMarkingDeque(traceable);
   }
 
-  /**
-   * Trace all wrappers of |t|.
-   *
-   * If you cannot use TraceWrapperMember & the corresponding traceWrappers()
-   * for some reason (e.g., due to sizeof(TraceWrapperMember)), you can use
-   * Member and |traceWrappersWithManualWriteBarrier()|. See below.
-   */
+  // Trace all wrappers of |t|.
+  //
+  // If you cannot use TraceWrapperMember & the corresponding traceWrappers()
+  // for some reason (e.g., due to sizeof(TraceWrapperMember)), you can use
+  // Member and |traceWrappersWithManualWriteBarrier()|. See below.
   template <typename T>
   void TraceWrappers(const TraceWrapperMember<T>& t) const {
     TraceWrappers(t.Get());
   }
 
-  /**
-   * Require all users of manual write barriers to make this explicit in their
-   * |traceWrappers| definition. Be sure to add
-   * |ScriptWrappableVisitor::writeBarrier(this, new_value)| after all
-   * assignments to the field. Otherwise, the objects may be collected
-   * prematurely.
-   */
+  // Require all users of manual write barriers to make this explicit in their
+  // |traceWrappers| definition. Be sure to add
+  // |ScriptWrappableVisitor::writeBarrier(this, new_value)| after all
+  // assignments to the field. Otherwise, the objects may be collected
+  // prematurely.
   template <typename T>
   void TraceWrappersWithManualWriteBarrier(const Member<T>& t) const {
     TraceWrappers(t.Get());
   }
   template <typename T>
   void TraceWrappersWithManualWriteBarrier(const WeakMember<T>& t) const {
     TraceWrappers(t.Get());
   }
   template <typename T>
   void TraceWrappersWithManualWriteBarrier(const T* traceable) const {
@@ skipping 30 matching lines @@
   virtual bool PushToMarkingDeque(
       void (*trace_wrappers_callback)(const WrapperVisitor*, const void*),
       HeapObjectHeader* (*heap_object_header_callback)(const void*),
       void (*missed_write_barrier_callback)(void),
       const void*) const = 0;
 };
 
 }  // namespace blink
 
 #endif  // WrapperVisitor_h