Bug 1547138 - followup: Let the compiler generate more IntrinsicSize functions as suggested by TYLin.
authorEmilio Cobos Álvarez <emilio@crisal.io>
Fri, 26 Apr 2019 02:18:42 +0200
changeset 530220 ee95a11e12edabbf8f75fb17517524ef043e265d
parent 530219 62da064b1b8f2274df270f8f2de784e9b4d0302d
child 530221 894d95f52dc2387e9112ac682f2af7f35309bad9
push id11265
push userffxbld-merge
push dateMon, 13 May 2019 10:53:39 +0000
treeherdermozilla-beta@77e0fe8dbdd3 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1547138
milestone68.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 1547138 - followup: Let the compiler generate more IntrinsicSize functions as suggested by TYLin.
layout/generic/nsIFrame.h
--- a/layout/generic/nsIFrame.h
+++ b/layout/generic/nsIFrame.h
@@ -473,22 +473,16 @@ struct IntrinsicSize {
   Maybe<nscoord> width;
   Maybe<nscoord> height;
 
   IntrinsicSize() = default;
 
   IntrinsicSize(nscoord aWidth, nscoord aHeight)
       : width(Some(aWidth)), height(Some(aHeight)) {}
 
-  IntrinsicSize(const IntrinsicSize& rhs) = default;
-  IntrinsicSize& operator=(const IntrinsicSize& rhs) {
-    width = rhs.width;
-    height = rhs.height;
-    return *this;
-  }
   bool operator==(const IntrinsicSize& rhs) {
     return width == rhs.width && height == rhs.height;
   }
   bool operator!=(const IntrinsicSize& rhs) { return !(*this == rhs); }
 };
 
 // Pseudo bidi embedding level indicating nonexistence.
 static const nsBidiLevel kBidiLevelNone = 0xff;