Rewrite to use Web IDL, and generally modernize

.editorconfig

@@ -7,15 +7,19 @@ charset = utf-8
 indent_size = 2
 indent_style = space
 trim_trailing_whitespace = true
+max_line_length = 100
 
-[*.{js,bs}]
+[Makefile]
+indent_style = tab
+
+[*.bs]
+indent_size = 1
+
+[*.py]
+indent_size = 4
+
+[*.js]
 max_line_length = 120
 
 [.gitmodules]
 indent_style = tab
-
-[Makefile]
-indent_style = tab
-
-[.travis.yml]
-indent_size = 2

.gitignore

@@ -4,6 +4,3 @@
 /deploy_key.pub
 /index.html
 /review.sh
-/index.html.*
-/node_modules/
-/npm-debug.log

.pr-preview.json

@@ -1,15 +1,9 @@
 {
-    "src_file": "index.bs",
-    "type": "bikeshed",
-    "params": {
-        "force": 1,
-        "md-status": "LS-PR",
-        "md-Text-Macro": "PR-NUMBER {{ pull_request.number }}"
-    },
-    "post_processing": {
-        "name": "emu-algify",
-        "options": {
-            "throwingIndicators": true
-        }
-    }
+  "src_file": "index.bs",
+  "type": "bikeshed",
+  "params": {
+    "force": 1,
+    "md-status": "LS-PR",
+    "md-Text-Macro": "PR-NUMBER {{ pull_request.number }}"
+  }
 }

.travis.yml

@@ -1,6 +1,5 @@
 language: node_js
 node_js: stable
-sudo: false
 
 env:
   global:
@@ -12,7 +11,6 @@ before_install:
 script:
   - npm test
   - cd ..
-  - npm install
   - make deploy
 
 branches:

Makefile

@@ -2,17 +2,25 @@ SHELL=/bin/bash -o pipefail
 .PHONY: local remote deploy review
 
 remote: index.bs
-	curl https://api.csswg.org/bikeshed/ -f -F [email protected] > index.html.postbs -F md-Text-Macro="SNAPSHOT-LINK LOCAL COPY"
-	node_modules/.bin/emu-algify --throwing-indicators < index.html.postbs > index.html
+	@ (HTTP_STATUS=$$(curl https://api.csswg.org/bikeshed/ \
+	                       --output index.html \
+	                       --write-out "%{http_code}" \
+	                       --header "Accept: text/plain, text/html" \
+	                       -F die-on=warning \
+												 -F md-Text-Macro="COMMIT-SHA LOCAL COPY" \
+	                       -F [email protected]) && \
+	[[ "$$HTTP_STATUS" -eq "200" ]]) || ( \
+		echo ""; cat index.html; echo ""; \
+		rm -f index.html; \
+		exit 22 \
+	);
 
 local: index.bs
-	bikeshed spec index.bs index.html.postbs --md-Text-Macro="SNAPSHOT-LINK LOCAL COPY"
-	node_modules/.bin/emu-algify --throwing-indicators < index.html.postbs > index.html
+	bikeshed spec index.bs index.html --md-Text-Macro="COMMIT-SHA LOCAL COPY"
 
 deploy: index.bs
 	curl --remote-name --fail https://resources.whatwg.org/build/deploy.sh
 	EXTRA_FILES="demos/* demos/**/*" \
-	POST_BUILD_STEP='node_modules/.bin/emu-algify --throwing-indicators < "$$DIR/index.html" > "$$DIR/index.html.tmp"; mv "$$DIR/index.html.tmp" "$$DIR/index.html"' \
 	bash ./deploy.sh
 
 review: index.bs

README.md

@@ -37,10 +37,6 @@ implementation in order to pass those tests.
 
 ## Building "locally"
 
-This standard requires a recent version of [Node.js](https://nodejs.org/en/) to be installed as a
-prerequisite. Once that's done, you'll need to do a one-time run of `npm install` to set up our
-tooling.
-
 For quick local iteration, run `make`. To verify your changes locally, run `make deploy`. See more
 in the
 [WHATWG Contributor Guidelines](https://github.com/whatwg/meta/blob/master/CONTRIBUTING.md#building).

reference-implementation/.eslintignore

@@ -1,2 +1,3 @@
 web-platform-tests/
 coverage/
+generated/

reference-implementation/.eslintrc.json

@@ -3,19 +3,15 @@
   "env": {
     "node": true,
     "browser": true,
-    "es6": true
+    "es2020": true
   },
   "parserOptions": {
-    "ecmaVersion": 2018
+    "ecmaVersion": 2020
   },
   "globals": {
-    "ReadableStream": false,
-    "WritableStream": false,
-    "TransformStream": false,
-    "ByteLengthQueuingStrategy": false,
-    "CountQueuingStrategy": false,
     "GCController": false,
-    "gc": false
+    "gc": false,
+    "globalThis": false
   },
   "rules": {
     // Possible errors

reference-implementation/.gitignore

@@ -4,3 +4,4 @@ package-lock.json
 
 .nyc_output/
 coverage/
+generated/

reference-implementation/compile-idl.js

@@ -0,0 +1,22 @@
+'use strict';
+/* eslint-disable no-console, no-process-exit */
+const { mkdirSync } = require('fs');
+const path = require('path');
+const Transformer = require('webidl2js');
+
+const input = path.resolve(__dirname, './lib');
+const output = path.resolve(__dirname, './generated');
+
+mkdirSync(output, { recursive: true });
+
+const transformer = new Transformer({
+  implSuffix: '-impl',
+  suppressErrors: true // until https://github.com/jsdom/webidl2js/pull/123 lands
+});
+
+transformer.addSource(input, input);
+transformer.generate(output)
+  .catch(err => {
+    console.error(err.stack);
+    process.exit(1);
+  });

reference-implementation/lib/ByteLengthQueuingStrategy-impl.js

@@ -0,0 +1,26 @@
+'use strict';
+
+exports.implementation = class ByteLengthQueuingStrategyImpl {
+  constructor(globalObject, [{ highWaterMark }]) {
+    this._globalObject = globalObject;
+    this.highWaterMark = highWaterMark;
+  }
+
+  get size() {
+    initializeSizeFunction(this._globalObject);
+    return sizeFunctionWeakMap.get(this._globalObject);
+  }
+};
+
+const sizeFunctionWeakMap = new WeakMap();
+function initializeSizeFunction(globalObject) {
+  if (sizeFunctionWeakMap.has(globalObject)) {
+    return;
+  }
+
+  // We need to set the 'name' property:
+  // eslint-disable-next-line prefer-arrow-callback
+  sizeFunctionWeakMap.set(globalObject, function size(chunk) {
+    return chunk.byteLength;
+  });
+}

reference-implementation/lib/ByteLengthQueuingStrategy.webidl

@@ -0,0 +1,7 @@
+[Exposed=(Window,Worker,Worklet)]
+interface ByteLengthQueuingStrategy {
+  constructor(QueuingStrategyInit init);
+
+  readonly attribute unrestricted double highWaterMark;
+  readonly attribute Function size;
+};

reference-implementation/lib/CountQueuingStrategy-impl.js

@@ -0,0 +1,26 @@
+'use strict';
+
+exports.implementation = class CountQueuingStrategyImpl {
+  constructor(globalObject, [{ highWaterMark }]) {
+    this._globalObject = globalObject;
+    this.highWaterMark = highWaterMark;
+  }
+
+  get size() {
+    initializeSizeFunction(this._globalObject);
+    return sizeFunctionWeakMap.get(this._globalObject);
+  }
+};
+
+const sizeFunctionWeakMap = new WeakMap();
+function initializeSizeFunction(globalObject) {
+  if (sizeFunctionWeakMap.has(globalObject)) {
+    return;
+  }
+
+  // We need to set the 'name' property:
+  // eslint-disable-next-line prefer-arrow-callback
+  sizeFunctionWeakMap.set(globalObject, function size() {
+    return 1;
+  });
+}

reference-implementation/lib/CountQueuingStrategy.webidl

@@ -0,0 +1,7 @@
+[Exposed=(Window,Worker,Worklet)]
+interface CountQueuingStrategy {
+  constructor(QueuingStrategyInit init);
+
+  readonly attribute unrestricted double highWaterMark;
+  readonly attribute Function size;
+};

reference-implementation/lib/QueuingStrategy.webidl

@@ -0,0 +1,6 @@
+dictionary QueuingStrategy {
+  unrestricted double highWaterMark;
+  QueuingStrategySize size;
+};
+
+callback QueuingStrategySize = unrestricted double (optional any chunk);

reference-implementation/lib/QueuingStrategyInit.webidl

@@ -0,0 +1,3 @@
+dictionary QueuingStrategyInit {
+  required unrestricted double highWaterMark;
+};