From 14a891c4b0836f1843d9f68331fb7873a46ee2b7 Mon Sep 17 00:00:00 2001
From: Jinho Bang <zino@chromium.org>
Date: Fri, 15 Dec 2017 22:45:24 +0900
Subject: [PATCH] Use `interface mixins` instead of `[NoInterfaceObject]`

WebIDL recently introduced dedicated syntax for mixins[1]. So, we can
replace `[NoInterfaceObject]` and `implements` with `interface mixin` and
`includes`.

This following interface is impacted by this change:
  - NavigatorStorage

This fixes #53 issue.

[1] https://github.com/heycam/webidl/commit/45e8173d40ddff8dcf81697326e094bcf8b92920
---
 storage.bs | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/storage.bs b/storage.bs
index cfc074c..9346da6 100644
--- a/storage.bs
+++ b/storage.bs
@@ -221,14 +221,12 @@ user agent should alert the user and offer a way to clear <a>persistent buckets<
 <h2 id=api>API</h2>
 
 <pre class=idl>
-[SecureContext,
- NoInterfaceObject,
- Exposed=(Window,Worker)]
-interface NavigatorStorage {
+[SecureContext]
+interface mixin NavigatorStorage {
   readonly attribute StorageManager storage;
 };
-Navigator implements NavigatorStorage;
-WorkerNavigator implements NavigatorStorage;
+Navigator includes NavigatorStorage;
+WorkerNavigator includes NavigatorStorage;
 </pre>
 
 Each <a>environment settings object</a> has an associated {{StorageManager}} object.