From 5eb2899146ac87a5973ff2826d2bb483b19544b2 Mon Sep 17 00:00:00 2001
From: Anne van Kesteren <annevk@annevk.nl>
Date: Tue, 27 Feb 2018 16:05:40 +0100
Subject: [PATCH 1/2] Use SetIntegrityLevel instead of Object.freeze()

Verbose, but clear and correct.

Fixes #74.
---
 notifications.bs | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/notifications.bs b/notifications.bs
index 8d57720..c59327c 100644
--- a/notifications.bs
+++ b/notifications.bs
@@ -909,13 +909,13 @@ return the <a>notification</a>'s <a for=notification>require interaction prefere
 result of the following steps:
 
 <ol>
-  <li><p>Let <var>frozenActions</var> be an empty list of type {{NotificationAction}}.
+  <li><p>Let <var>frozenActions</var> be a new <a for=/>list</a>.
 
   <li><p>For each <var>entry</var> in the <a>notification</a>'s list of
   <a for=notification>actions</a>, perform the following steps:
 
   <ol>
-    <li><p>Let <var>action</var> be a new {{NotificationAction}}.
+    <li><p>Let <var>action</var> be a new {{NotificationAction}} <a for=/>dictionary</a>.
 
     <li><p>Set <var>action</var>'s {{NotificationAction/action}} to
     <var>entry</var>'s <a for=action>name</a>.
@@ -926,15 +926,15 @@ result of the following steps:
     <li><p>Set <var>action</var>'s {{NotificationAction/icon}} to
     <var>entry</var>'s <a for=action>icon URL</a>.
 
-    <!-- XXX IDL dictionaries are usually returned by value, so don't need to be
-         immutable. But FrozenArray reifies the dictionaries to mutable JS
-         objects accessed by reference, so we explicitly freeze them.
-         It would be better to do this with IDL primitives instead of JS - see
-         https://www.w3.org/Bugs/Public/show_bug.cgi?id=29004 -->
-    <li><p>Call <a lt="freeze">Object.freeze</a> on <var>action</var>, to
-    prevent accidental mutation by scripts.
+    <li><p>Let <var>jsAction</var> be <var>action</var>, <a>converted to an ECMAScript value</a>.
 
-    <li><p>Append <var>action</var> to <var>frozenActions</var>.
+    <li><p>Perform <a>SetIntegrityLevel</a>(<var>jsAction</var>, "frozen").
+
+    <li><p>Let <var>idlAction</var> be <var>jsAction</var>, <a>converted to an IDL value</a>.
+    <!-- See https://www.w3.org/Bugs/Public/show_bug.cgi?id=29004 about replacing the above 3 steps
+         with something simpler. -->
+
+    <li><p><a for=list>Append</a> <var>idlAction</var> to <var>frozenActions</var>.
   </ol>
 
   <li><p><a>Create a frozen array</a> from <var>frozenActions</var>.
@@ -1224,8 +1224,8 @@ urlPrefix: https://w3c.github.io/vibration/
         text: validate and normalize
     urlPrefix: #idl-def-; type: interface
         text: VibratePattern; url: vibratepattern
-urlPrefix: https://tc39.github.io/ecma262/#sec-object.; type: dfn
-    text: freeze
+urlPrefix: https://tc39.github.io/ecma262/#sec-; spec: ECMA-262; type: abstract-op
+    text: SetIntegrityLevel
 </pre>
 
 <pre class="biblio">

From 9e4392db003ddd5bfd9e1ef4441e185df515f148 Mon Sep 17 00:00:00 2001
From: Anne van Kesteren <annevk@annevk.nl>
Date: Tue, 27 Feb 2018 16:08:10 +0100
Subject: [PATCH 2/2] nit

---
 notifications.bs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/notifications.bs b/notifications.bs
index c59327c..8c63c5c 100644
--- a/notifications.bs
+++ b/notifications.bs
@@ -928,7 +928,7 @@ result of the following steps:
 
     <li><p>Let <var>jsAction</var> be <var>action</var>, <a>converted to an ECMAScript value</a>.
 
-    <li><p>Perform <a>SetIntegrityLevel</a>(<var>jsAction</var>, "frozen").
+    <li><p>Perform <a abstract-op>SetIntegrityLevel</a>(<var>jsAction</var>, "frozen").
 
     <li><p>Let <var>idlAction</var> be <var>jsAction</var>, <a>converted to an IDL value</a>.
     <!-- See https://www.w3.org/Bugs/Public/show_bug.cgi?id=29004 about replacing the above 3 steps