Bug 775624 Part 23 - Remove constructor and all operators related to uint32_t. r=dholbert
authorTing-Yu Lin <tlin@mozilla.com>
Tue, 14 Feb 2017 18:01:11 +0800
changeset 373414 672ba79a78402d839579d9a71d865ec2d85cef0a
parent 373413 cb33bd098140c9cef18c8716853bb6c5e29190ac
child 373415 3095d085ef20df5588f4ee649d14c8e8762f1a72
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdholbert
bugs775624
milestone54.0a1
Bug 775624 Part 23 - Remove constructor and all operators related to uint32_t. r=dholbert MozReview-Commit-ID: FEVYMhcpPS0
layout/generic/nsIFrame.h
--- a/layout/generic/nsIFrame.h
+++ b/layout/generic/nsIFrame.h
@@ -196,18 +196,17 @@ enum nsSpread {
 
 //----------------------------------------------------------------------
 // Reflow status returned by the Reflow() methods.
 class nsReflowStatus final {
   using StyleClear = mozilla::StyleClear;
 
 public:
   nsReflowStatus()
-    : mStatus(0)
-    , mBreakType(StyleClear::None)
+    : mBreakType(StyleClear::None)
     , mIncomplete(false)
     , mOverflowIncomplete(false)
     , mNextInFlowNeedsReflow(false)
     , mTruncated(false)
     , mInlineBreak(false)
     , mInlineBreakAfter(false)
     , mFirstLetterComplete(false)
   {}
@@ -219,53 +218,16 @@ public:
     mOverflowIncomplete = false;
     mNextInFlowNeedsReflow = false;
     mTruncated = false;
     mInlineBreak = false;
     mInlineBreakAfter = false;
     mFirstLetterComplete = false;
   }
 
-  nsReflowStatus(uint32_t aStatus)
-    : mStatus(aStatus)
-  {}
-
-  uint32_t& operator=(uint32_t aRhs) {
-    mStatus = aRhs;
-    return mStatus;
-  }
-
-  uint32_t operator&(uint32_t aRhs) const {
-    return mStatus & aRhs;
-  }
-
-  uint32_t operator&=(uint32_t aRhs) {
-    return mStatus = mStatus & aRhs;
-  }
-
-  uint32_t operator|(uint32_t aRhs) const {
-    return mStatus | aRhs;
-  }
-
-  uint32_t operator|=(uint32_t aRhs) {
-    return mStatus = mStatus | aRhs;
-  }
-
-  uint32_t operator>>(uint32_t aRhs) const {
-    return mStatus >> aRhs;
-  }
-
-  bool operator==(uint32_t aRhs) const {
-    return mStatus == aRhs;
-  }
-
-  bool operator!=(uint32_t aRhs) const {
-    return !(*this == aRhs);
-  }
-
   // Return true if all flags are cleared.
   bool IsEmpty() const {
     return (!mIncomplete &&
             !mOverflowIncomplete &&
             !mNextInFlowNeedsReflow &&
             !mTruncated &&
             !mInlineBreak &&
             !mInlineBreakAfter &&
@@ -359,18 +321,16 @@ public:
   }
 
   // mFirstLetterComplete bit flag means the break was induced by
   // completion of a first-letter.
   bool FirstLetterComplete() const { return mFirstLetterComplete; }
   void SetFirstLetterComplete() { mFirstLetterComplete = true; }
 
 private:
-  uint32_t mStatus;
-
   StyleClear mBreakType;
 
   // Frame completion status bit flags.
   bool mIncomplete : 1;
   bool mOverflowIncomplete : 1;
   bool mNextInFlowNeedsReflow : 1;
   bool mTruncated : 1;