Bug 1370037 - Remove an unnecessary friend declaration in FrameProperties. r=jfkthame
authorMats Palmgren <mats@mozilla.com>
Sun, 04 Jun 2017 16:45:15 +0200
changeset 412713 e429006e7579931f2f70768e0c1f016b1d270e2e
parent 412687 9d55c2ccf868c6f3e5c75b3bf94069c1795881a1
child 412714 1005b7702f1e6ba7679d73365407aa20fe10e08c
push id1490
push usermtabara@mozilla.com
push dateMon, 31 Jul 2017 14:08:16 +0000
treeherdermozilla-release@70e32e6bf15e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjfkthame
bugs1370037
milestone55.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1370037 - Remove an unnecessary friend declaration in FrameProperties. r=jfkthame MozReview-Commit-ID: F6OfF8GsPhS
layout/base/FrameProperties.h
--- a/layout/base/FrameProperties.h
+++ b/layout/base/FrameProperties.h
@@ -301,18 +301,16 @@ public:
     // As for the PropertyValue entries: we don't need to measure the mProperty
     // field of because it always points to static memory, and we can't measure
     // mValue because the type is opaque.
     // XXX Can we do better, e.g. with a method on the descriptor?
     return mProperties.ShallowSizeOfExcludingThis(aMallocSizeOf);
   }
 
 private:
-  friend class ::nsIFrame;
-
   // Prevent copying of FrameProperties; we should always return/pass around
   // references to it, not copies!
   FrameProperties(const FrameProperties&) = delete;
   FrameProperties& operator=(const FrameProperties&) = delete;
 
   inline void
   SetInternal(UntypedDescriptor aProperty, void* aValue,
               const nsIFrame* aFrame);