BREAKING CHANGE(core/webidl): deprecate serializers

w3c · Jun 27, 2017 · 00df45a · 00df45a
1 parent d845633
commit 00df45a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 36 deletions.
diff --git a/js/core/templates/webidl-contiguous/serializer.html b/js/core/templates/webidl-contiguous/serializer.html
diff --git a/package.json b/package.json
@@ -51,7 +51,7 @@
     "text": "github:requirejs/text",
     "uglify-es": "^3.0.20",
     "url-search-params": "^0.9.0",
-    "webidl2": "^3.0.2"
+    "webidl2": "^4.0.0"
   },
   "scripts": {
     "babel:build": "babel src -d js --source-maps",

diff --git a/src/core/webidl.js b/src/core/webidl.js
@@ -28,7 +28,6 @@ var idlMaplikeTmpl = tmpls["maplike.html"];
 var idlMethodTmpl = tmpls["method.html"];
 var idlMultiLineCommentTmpl = tmpls["multiline-comment.html"];
 var idlParamTmpl = tmpls["param.html"];
-var idlSerializerTmpl = tmpls["serializer.html"];
 var idlTypedefTmpl = tmpls["typedef.html"];
 // TODO: make these linkable somehow.
 // https://github.com/w3c/respec/issues/999
@@ -426,7 +425,6 @@ const idlKeywords = new Set([
   "RegExp",
   "required",
   "sequence",
-  "serializer",
   "setlike",
   "setter",
   "short",
@@ -457,7 +455,6 @@ const argumentNameKeyword = new Set([
   "maplike",
   "partial",
   "required",
-  "serializer",
   "setlike",
   "setter",
   "static",
@@ -646,7 +643,6 @@ function writeInterfaceDefinition(opt, callback) {
   obj.members.forEach(function(it) {
     if (
       typeIsWhitespace(it.type) ||
-      it.type === "serializer" ||
       it.type === "maplike" ||
       it.type === "iterable"
     ) {
@@ -671,8 +667,6 @@ function writeInterfaceDefinition(opt, callback) {
           return writeMethod(ch, maxMeth, indent + 1);
         case "const":
           return writeConst(ch, maxConst, indent + 1);
-        case "serializer":
-          return writeSerializer(ch, indent + 1);
         case "maplike":
           return writeMaplike(ch, indent + 1);
         case "iterable":
@@ -749,7 +743,6 @@ function writeMethod(meth, max, indent) {
     "setter",
     "deleter",
     "legacycaller",
-    "serializer",
     "stringifier",
   ];
   var special = "";
@@ -824,22 +817,6 @@ function writeMultiLineComment(comment, indent) {
   });
 }
 
-function writeSerializer(serializer, indent) {
-  var values = "";
-  if (serializer.patternMap) {
-    values = "{" + serializer.names.join(", ") + "}";
-  } else if (serializer.patternList) {
-    values = "[" + serializer.patternList.join(", ") + "]";
-  } else if (serializer.name) {
-    values = serializer.name;
-  }
-  return idlSerializerTmpl({
-    obj: serializer,
-    indent: indent,
-    values: values,
-  });
-}
-
 function writeMaplike(maplike, indent) {
   var qualifiers = "";
   if (maplike.readonly) qualifiers += "readonly ";
@@ -947,8 +924,7 @@ function linkDefinitions(parse, definitionMap, parent, idlElem) {
           defn.setter ||
           defn.deleter ||
           defn.legacycaller ||
-          defn.stringifier ||
-          defn.serializer
+          defn.stringifier
         ) {
           name = "";
         }
@@ -978,12 +954,6 @@ function linkDefinitions(parse, definitionMap, parent, idlElem) {
         defn.idlId =
           "idl-def-" + parent.toLowerCase() + "-" + name.toLowerCase();
         break;
-      case "serializer":
-        name = "serializer";
-        defn.idlId =
-          "idl-def-" + parent.toLowerCase() + "-" + name.toLowerCase();
-        break;
-
       case "implements":
       case "ws":
       case "ws-pea":
@@ -1170,7 +1140,7 @@ export function run(conf, doc, cb) {
     $df.attr({ id: this.id });
     $df
       .find(
-        ".idlAttribute,.idlCallback,.idlConst,.idlDictionary,.idlEnum,.idlException,.idlField,.idlInterface,.idlMember,.idlMethod,.idlSerializer,.idlMaplike,.idlIterable,.idlTypedef"
+        ".idlAttribute,.idlCallback,.idlConst,.idlDictionary,.idlEnum,.idlException,.idlField,.idlInterface,.idlMember,.idlMethod,.idlMaplike,.idlIterable,.idlTypedef"
       )
       .each(function() {
         var elem = $(this);