From dd565106af51a8e604b7ba7dcb53ab00f55ff4cd Mon Sep 17 00:00:00 2001
From: Joshua Bell <jsbell@chromium.org>
Date: Thu, 6 Jun 2019 17:27:33 -0700
Subject: [PATCH] Editorial: Fix _includes link, add note.

"includes" is reserved word in WebIDL, so a method with that name
needs to be escaped with a leading underscore in IDL fragments.
This confuses Bikeshed (tabatkins/bikeshed#1489) so manually
correct the link. Also, the 2.0 version of the IDB spec had a note
which gave context for this; transplant that to the 3.0 version.

Resolves #270
---
 index.bs | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/index.bs b/index.bs
index e30cafe..9d4b599 100644
--- a/index.bs
+++ b/index.bs
@@ -4361,6 +4361,10 @@ interface IDBKeyRange {
 };
 </xmp>
 
+<aside class=note>
+Note: When mapping the `_includes` identifier from [[WEBIDL]] to ECMAScript, the leading U+005F LOW LINE ("_") character is removed. A leading "_" is used to escape the identifier from looking like a reserved word (in this case, the `includes` keyword).
+</aside>
+
 <div class=note>
   <dl class=domintro>
     <dt><var>range</var> . {{IDBKeyRange/lower}}</dt>
@@ -4517,7 +4521,7 @@ The <dfn method for=IDBKeyRange>bound(|lower|, |upper|, |lowerOpen|,
 
 <div class=algorithm>
 
-The <dfn method for=IDBKeyRange>includes(|key|)</dfn> method, when
+The <dfn method for=IDBKeyRange lt="_includes(key)|includes(key)">includes(|key|)</dfn> method, when
 invoked, must run these steps:
 
 1. Let |k| be the result of running the steps to [=convert a