Fix a crash with empty @contents blocks.

Steinar H. Gunderson · chromium-wpt-export-bot · commit bc1c54d8d486 · 2025-09-05T06:30:39.000-07:00
Also fixed a typo in the test while we were on it. Fixed: 442735990 Change-Id: I4c448272bd9bfeef04ba0be60d9ccef751516d85 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6919119 Reviewed-by: Anders Hartvoll Ruud <andruud@chromium.org> Auto-Submit: Steinar H Gunderson <sesse@chromium.org> Commit-Queue: Steinar H Gunderson <sesse@chromium.org> Cr-Commit-Position: refs/heads/main@{#1511504}
diff --git a/css/css-mixins/contents-rule.html b/css/css-mixins/contents-rule.html
@@ -46,9 +46,17 @@
         @contents { color: red !important; }
         color: green;
       }
-      #e4 {
+      #e5 {
         @apply --m5 { color: red !important; }
       }
+
+      @mixin --m6(@contents) {
+        @contents { }
+        color: green;
+      }
+      #e6 {
+        @apply --m6;
+      }
     </style>
   </head>
   <body>
@@ -57,6 +65,7 @@
     <div class="a b" id="e3">This text should be green.</div>
     <div class="c d" id="e4">This text should be green.</div>
     <div id="e5">This text should be green.</div>
+    <div id="e6">This text should be green.</div>
   <script>
     test(() => {
       let target = document.getElementById('e1');
@@ -82,6 +91,11 @@
       let target = document.getElementById('e5');
       assert_equals(getComputedStyle(target).color, 'rgb(0, 128, 0)');
     }, '@contents is ignored if there is no @contents parameter');
+
+    test(() => {
+      let target = document.getElementById('e6');
+      assert_equals(getComputedStyle(target).color, 'rgb(0, 128, 0)');
+    }, 'Empty @contents block is allowed, but does nothing');
   </script>
   </body>
 </html>
