W3C home > Mailing lists > Public > public-css-archive@w3.org > March 2019

Re: [csswg-drafts] [css-variables] Token sequence size limit (#3733)

From: Emilio Cobos Álvarez via GitHub <sysbot+gh@w3.org>
Date: Fri, 15 Mar 2019 22:38:43 +0000
To: public-css-archive@w3.org
Message-ID: <issue_comment.created-473462842-1552689522-sysbot+gh@w3.org>
I'm pretty sure that's not the case. I can reproduce the timeout if I check out your test locally, but that's because the harness expects at least one test to run.

As soon as I apply:

diff --git a/css/billion-laughs.html b/css/billion-laughs.html
index 0d80bbb7d7..648d750e09 100644
--- a/css/billion-laughs.html
+++ b/css/billion-laughs.html
@@ -46,3 +46,9 @@ an impl that correctly implements the mitigation described in the spec
 will instead execute this no-op page
 and thus "pass" it by default.
+let t = async_test("Foo");
+onload = t.step_func_done(function() {
+  document.documentElement.offsetTop; // ensure layout runs.

The test passes just fine as expected. @jgraham do you know what's the expected behavior of a test without a test function?

GitHub Notification of comment by emilio
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/3733#issuecomment-473462842 using your GitHub account
Received on Friday, 15 March 2019 22:38:44 UTC

This archive was generated by hypermail 2.4.0 : Thursday, 24 March 2022 20:26:58 UTC