Bug 1507604 - Remove unused GetSerial methods. r=smaug

Differential Revision: https://phabricator.services.mozilla.com/D12137

--HG--
extra : moz-landing-system : lando

dom/base/nsGlobalWindowInner.cpp

@@ -918,8 +918,10 @@ nsGlobalWindowInner::nsGlobalWindowInner(nsGlobalWindowOuter *aOuterWindow)
     mHasSeenGamepadInput(false),
     mSuspendDepth(0),
     mFreezeDepth(0),
-    mFocusMethod(0),
+#ifdef DEBUG
     mSerial(0),
+#endif
+    mFocusMethod(0),
     mIdleRequestCallbackCounter(1),
     mIdleRequestExecutor(nullptr),
     mDialogAbuseCount(0),
@@ -977,8 +979,6 @@ nsGlobalWindowInner::nsGlobalWindowInner(nsGlobalWindowOuter *aOuterWindow)
   // to create the entropy collector, so we should
   // try to get one until we succeed.
 
-  mSerial = nsContentUtils::InnerOrOuterWindowCreated();
-
   static bool sFirstTime = true;
   if (sFirstTime) {
     sFirstTime = false;
@@ -1000,6 +1000,8 @@ nsGlobalWindowInner::nsGlobalWindowInner(nsGlobalWindowOuter *aOuterWindow)
   }
 
 #ifdef DEBUG
+  mSerial = nsContentUtils::InnerOrOuterWindowCreated();
+
   if (!PR_GetEnv("MOZ_QUIET")) {
     printf_stderr("++DOMWINDOW == %d (%p) [pid = %d] [serial = %d] [outer = %p]\n",
                   nsContentUtils::GetCurrentInnerOrOuterWindowCount(),

dom/base/nsGlobalWindowInner.h

@@ -511,10 +511,6 @@ public:
   virtual void DisableOrientationChangeListener() override;
 #endif
 
-  virtual uint32_t GetSerial() override {
-    return mSerial;
-  }
-
   void AddSizeOfIncludingThis(nsWindowSizes& aWindowSizes) const;
 
   void NotifyIdleObserver(IdleObserverHolder* aIdleObserverHolder,
@@ -1427,11 +1423,13 @@ protected:
   uint32_t mSuspendDepth;
   uint32_t mFreezeDepth;
 
+#ifdef DEBUG
+  uint32_t mSerial;
+#endif
+
   // the method that was used to focus mFocusedNode
   uint32_t mFocusMethod;
 
-  uint32_t mSerial;
-
   // The current idle request callback handle
   uint32_t mIdleRequestCallbackCounter;
   IdleRequests mIdleRequestCallbacks;

dom/base/nsGlobalWindowOuter.cpp

@@ -842,8 +842,8 @@ nsGlobalWindowOuter::nsGlobalWindowOuter(uint64_t aWindowID)
     mAllowScriptsToClose(false),
     mTopLevelOuterContentWindow(false),
     mHasStorageAccess(false),
-    mSerial(0),
 #ifdef DEBUG
+    mSerial(0),
     mSetOpenerWindowCalled(false),
 #endif
     mCleanedUp(false),
@@ -868,9 +868,9 @@ nsGlobalWindowOuter::nsGlobalWindowOuter(uint64_t aWindowID)
   // to create the entropy collector, so we should
   // try to get one until we succeed.
 
+#ifdef DEBUG
   mSerial = nsContentUtils::InnerOrOuterWindowCreated();
 
-#ifdef DEBUG
   if (!PR_GetEnv("MOZ_QUIET")) {
     printf_stderr("++DOMWINDOW == %d (%p) [pid = %d] [serial = %d] [outer = %p]\n",
                   nsContentUtils::GetCurrentInnerOrOuterWindowCount(),

dom/base/nsGlobalWindowOuter.h

@@ -490,10 +490,6 @@ public:
                              bool aBlocked,
                              nsIURI* aURIHint) override;
 
-  virtual uint32_t GetSerial() override {
-    return mSerial;
-  }
-
   void AddSizeOfIncludingThis(nsWindowSizes& aWindowSizes) const;
 
   void AllowScriptsToClose()
@@ -1133,9 +1129,9 @@ protected:
 
   nsCOMPtr<nsIPrincipal> mDocumentPrincipal;
 
+#ifdef DEBUG
   uint32_t mSerial;
 
-#ifdef DEBUG
   bool mSetOpenerWindowCalled;
   nsCOMPtr<nsIURI> mLastOpenedURI;
 #endif

dom/base/nsPIDOMWindow.h

@@ -568,13 +568,6 @@ public:
    */
   virtual void SetHasGamepadEventListener(bool aHasGamepad = true) = 0;
 
-  /**
-   * NOTE! This function *will* be called on multiple threads so the
-   * implementation must not do any AddRef/Release or other actions that will
-   * mutate internal state.
-   */
-  virtual uint32_t GetSerial() = 0;
-
   /**
    * Return the window id of this window
    */
@@ -1099,13 +1092,6 @@ public:
    */
   virtual nsresult SetArguments(nsIArray *aArguments) = 0;
 
-  /**
-   * NOTE! This function *will* be called on multiple threads so the
-   * implementation must not do any AddRef/Release or other actions that will
-   * mutate internal state.
-   */
-  virtual uint32_t GetSerial() = 0;
-
   /**
    * Return the window id of this window
    */