Make HeapObjectHeader::checkHeader private.

third_party/WebKit/Source/platform/heap/Member.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 Member_h
 #define Member_h
 
 #include "wtf/Allocator.h"
 #include "wtf/HashFunctions.h"
 #include "wtf/HashTraits.h"
@@ skipping 137 matching lines @@
     // TODO(haraken): What we really want to check here is that the pointer
     // is a traceable object. In other words, the pointer is either of:
     //
     //   (a) a pointer to the head of an on-heap object.
     //   (b) a pointer to the head of an on-heap mixin object.
     //
     // We can check it by calling ThreadHeap::isHeapObjectAlive(m_raw),
     // but we cannot call it here because it requires to include T.h.
     // So we currently only try to implement the check for (a), but do
     // not insist that T's definition is in scope.
-    if (IsFullyDefined<T>::value && !IsGarbageCollectedMixin<T>::value)
-      HeapObjectHeader::fromPayload(m_raw)->checkHeader();
+    if (IsFullyDefined<T>::value && !IsGarbageCollectedMixin<T>::value) {
+      // |fromPayload| checks the integrity of the |HeapObjectHeader|.
+      (void)HeapObjectHeader::fromPayload(m_raw);
+    }
 #endif
 #endif
   }
 
   void saveCreationThreadState() {
 #if DCHECK_IS_ON()
     if (tracenessConfiguration == TracenessMemberConfiguration::Untraced) {
       m_creationThreadState = nullptr;
     } else {
       m_creationThreadState = ThreadState::current();
@@ skipping 370 matching lines @@
 
 template <typename T>
 struct IsTraceable<blink::TraceWrapperMember<T>> {
   STATIC_ONLY(IsTraceable);
   static const bool value = true;
 };
 
 }  // namespace WTF
 
 #endif  // Member_h