From 64a9820c62e8d0bef70a329c6c4784a13a910826 Mon Sep 17 00:00:00 2001
From: Molnar Sandor <smolnar@mozilla.com>
Date: Thu, 7 Apr 2022 03:02:28 +0300
Subject: [PATCH] Backed out changeset 9cc08c2f20be (bug 239460) for causing
 build bustage. CLOSED TREE

---
 netwerk/cookie/nsICookie.idl     |  6 +++---
 xpcom/idl-parser/xpidl/header.py | 14 ++++++++++----
 xpcom/system/nsIXULRuntime.idl   |  2 +-
 xpfe/appshell/AppWindow.cpp      |  4 +---
 4 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/netwerk/cookie/nsICookie.idl b/netwerk/cookie/nsICookie.idl
index 4d3ea2e023add..0b2e5e97bc9d9 100644
--- a/netwerk/cookie/nsICookie.idl
+++ b/netwerk/cookie/nsICookie.idl
@@ -15,9 +15,9 @@ typedef long nsCookiePolicy;
 
 [builtinclass, scriptable, uuid(adf0db5e-211e-45a3-be14-4486ac430a58)]
 interface nsICookie : nsISupports {
-    const int32_t SAMESITE_NONE   = 0;
-    const int32_t SAMESITE_LAX    = 1;
-    const int32_t SAMESITE_STRICT = 2;
+    const uint32_t SAMESITE_NONE   = 0;
+    const uint32_t SAMESITE_LAX    = 1;
+    const uint32_t SAMESITE_STRICT = 2;
 
     /**
      * the name of the cookie
diff --git a/xpcom/idl-parser/xpidl/header.py b/xpcom/idl-parser/xpidl/header.py
index c957ba845e792..a63ded29e14a5 100644
--- a/xpcom/idl-parser/xpidl/header.py
+++ b/xpcom/idl-parser/xpidl/header.py
@@ -495,16 +495,22 @@ def record_name(name):
             record_name(methodNativeName(m))
 
     def write_const_decls(g):
+        fd.write("  enum {\n")
+        enums = []
         for c in g:
             printComments(fd, c.doccomments, "  ")
-            fd.write(
-                "  static constexpr %(type)s %(name)s = %(value)s;\n"
+            basetype = c.basetype
+            value = c.getValue()
+            enums.append(
+                "    %(name)s = %(value)s%(signed)s"
                 % {
-                    "type": c.realtype.nativeType("in"),
                     "name": c.name,
-                    "value": c.getValue(),
+                    "value": value,
+                    "signed": (not basetype.signed) and "U" or "",
                 }
             )
+        fd.write(",\n".join(enums))
+        fd.write("\n  };\n\n")
 
     def write_cenum_decl(b):
         fd.write("  enum %s : uint%d_t {\n" % (b.basename, b.width))
diff --git a/xpcom/system/nsIXULRuntime.idl b/xpcom/system/nsIXULRuntime.idl
index c78d3c4565508..351b2766ca1f5 100644
--- a/xpcom/system/nsIXULRuntime.idl
+++ b/xpcom/system/nsIXULRuntime.idl
@@ -272,7 +272,7 @@ interface nsIXULRuntime : nsISupports
    * The current e10s-multi experiment number. Set dom.ipc.multiOptOut to (at
    * least) this to disable it until the next experiment.
    */
-  const int32_t E10S_MULTI_EXPERIMENT = 1;
+  const uint32_t E10S_MULTI_EXPERIMENT = 1;
 
   /**
    * If true, the accessibility service is running.
diff --git a/xpfe/appshell/AppWindow.cpp b/xpfe/appshell/AppWindow.cpp
index 50fd3e3d5adfc..d325b842ca4aa 100644
--- a/xpfe/appshell/AppWindow.cpp
+++ b/xpfe/appshell/AppWindow.cpp
@@ -1465,10 +1465,8 @@ bool AppWindow::UpdateWindowStateFromMiscXULAttributes() {
   if (!stateString.IsEmpty()) {
     nsresult errorCode;
     int32_t zLevel = stateString.ToInteger(&errorCode);
-    if (NS_SUCCEEDED(errorCode) && zLevel >= int32_t(lowestZ) &&
-        zLevel <= int32_t(highestZ)) {
+    if (NS_SUCCEEDED(errorCode) && zLevel >= lowestZ && zLevel <= highestZ)
       SetZLevel(zLevel);
-    }
   }
 
   return gotState;