From 826e89fc26938351ec46f23bbf2bff7edbf271f5 Mon Sep 17 00:00:00 2001
From: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
Date: Thu, 17 Oct 2019 10:50:55 +0200
Subject: [PATCH] Remove WakeLockEvent and WakeLockEventInit.

Per https://w3ctag.github.io/design-principles/#state-and-subclassing, we do
not really need a custom event class here.

`WakeLockEvent` has a single attribute, `lock`, that points to the event
target. That's exactly what `Event.target` also does, so we can just create
a simple `Event` instance and let the "fire an event" DOM algorithm set
`target` for us to achieve the same thing. In fact, it already does, so
before this commit we had

    ev.lock === ev.target

Fixes #238.
---
 index.html | 30 +++---------------------------
 1 file changed, 3 insertions(+), 27 deletions(-)

diff --git a/index.html b/index.html
index 05c55eb..2cd71a3 100644
--- a/index.html
+++ b/index.html
@@ -580,28 +580,6 @@ <h3>The <dfn>onrelease</dfn> attribute</h3>
         </aside>
       </section>
     </section>
-    <section data-dfn-for="WakeLockEvent" data-link-for="WakeLockEvent">
-      <h2>The <dfn>WakeLockEvent</dfn> interface</h2>
-      <pre class="idl">
-        [SecureContext, Exposed=(DedicatedWorker, Window)]
-        interface WakeLockEvent : Event {
-          constructor(DOMString type, WakeLockEventInit init);
-          readonly attribute WakeLockSentinel lock;
-        };
-
-        dictionary WakeLockEventInit : EventInit {
-          required WakeLockSentinel lock;
-        };
-      </pre>
-      <p><a>WakeLockEvent</a> objects indicate that an event has happened to a
-      given <a>WakeLockSentinel</a> object. A <a>WakeLockEvent</a> is
-      dispatched when a <a>WakeLockSentinel</a>'s wake lock is released.</p>
-      <section>
-        <h3>The <dfn>lock</dfn> attribute</h3>
-        <p>The <a>lock</a> attribute is a <a>WakeLockSentinel</a> object whose
-        wake lock has been released.</p>
-      </section>
-    </section>
     <section>
       <h2>
         Managing Wake Locks
@@ -824,10 +802,8 @@ <h3>
               </li>
             </ol>
           </li>
-          <li><a>Fire an event</a> named "`release`" at |lock| using
-          <a>WakeLockEvent</a> with its <a
-          data-link-for="WakeLockEvent">lock</a> attribute initialized to
-          |sentinel|.</li>
+          <li><a>Queue a task</a> to <a>fire an event</a> named "`release`" at
+          |lock|.</li>
         </ol>
       </section>
     </section>
@@ -883,7 +859,7 @@ <h2>
 
           const listener = (ev) => {
             checkbox.checked = false;
-            ev.lock.removeEventListener('release', listener);
+            ev.target.removeEventListener('release', listener);
           };
 
           lock.addEventListener('release', listener);