Backed out changeset 9cc08c2f20be (bug 239460) for causing build bust…

…age. CLOSED TREE
jamienicol · Apr 7, 2022 · 64a9820 · 64a9820
1 parent b3473b9
commit 64a9820
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 11 deletions.
diff --git 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
@@ -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
@@ -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
@@ -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;