Remove some dated Member friendships.

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 161 matching lines @@
       // collection iterator.
       DCHECK(m_creationThreadState || !m_raw);
     }
 #endif
   }
 
   T* m_raw;
 #if DCHECK_IS_ON()
   const ThreadState* m_creationThreadState;
 #endif
-
-  template <bool x,
-            WTF::WeakHandlingFlag y,
-            WTF::ShouldWeakPointersBeMarkedStrongly z,
-            typename U,
-            typename V>
-  friend struct CollectionBackingTraceTrait;
-  friend class Visitor;
 };
 
 // Members are used in classes to contain strong pointers to other oilpan heap
 // allocated objects.
 // All Member fields of a class must be traced in the class' trace method.
 // During the mark phase of the GC all live objects are marked as live and
 // all Member fields of a live object will be traced marked as live as well.
 template <typename T>
 class Member : public MemberBase<T, TracenessMemberConfiguration::Traced> {
   DISALLOW_NEW_EXCEPT_PLACEMENT_NEW();
@@ skipping 33 matching lines @@
   template <typename U>
   Member& operator=(U* other) {
     Parent::operator=(other);
     return *this;
   }
 
   Member& operator=(std::nullptr_t) {
     Parent::operator=(nullptr);
     return *this;
   }
-
- protected:
-  template <bool x,
-            WTF::WeakHandlingFlag y,
-            WTF::ShouldWeakPointersBeMarkedStrongly z,
-            typename U,
-            typename V>
-  friend struct CollectionBackingTraceTrait;
-  friend class Visitor;
 };
 
 // A checked version of Member<>, verifying that only same-thread references
 // are kept in the smart pointer. Intended to be used to diagnose unclean
 // thread reference usage in release builds. It simply exposes the debug-only
 // MemberBase<> checking we already have in place for select usage to diagnose
 // per-thread issues. Only intended used temporarily while diagnosing suspected
 // problems with cross-thread references.
 template <typename T>
 class SameThreadCheckedMember : public Member<T> {
@@ skipping 64 matching lines @@
     Parent::operator=(other);
     checkPointer();
     return *this;
   }
 
   SameThreadCheckedMember& operator=(std::nullptr_t) {
     Parent::operator=(nullptr);
     return *this;
   }
 
- protected:
-  template <bool x,
-            WTF::WeakHandlingFlag y,
-            WTF::ShouldWeakPointersBeMarkedStrongly z,
-            typename U,
-            typename V>
-  friend struct CollectionBackingTraceTrait;
-  friend class Visitor;
-
  private:
   void checkPointer() {
     if (!this->m_raw)
       return;
     // HashTable can store a special value (which is not aligned to the
     // allocation granularity) to Member<> to represent a deleted entry.
     // Thus we treat a pointer that is not aligned to the granularity
     // as a valid pointer.
     if (reinterpret_cast<intptr_t>(this->m_raw) % allocationGranularity)
       return;
@@ skipping 211 matching lines @@
 
 template <typename T>
 struct IsTraceable<blink::TraceWrapperMember<T>> {
   STATIC_ONLY(IsTraceable);
   static const bool value = true;
 };
 
 }  // namespace WTF
 
 #endif  // Member_h