From 41b3b90e8eb8b3072330531fb0938ad90436cdc4 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Thu, 15 Jul 2021 12:52:50 -0700
Subject: [PATCH 01/37] Add id member to manifest

---
 index.html | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 53 insertions(+)

diff --git a/index.html b/index.html
index bd461530e..6474703c4 100644
--- a/index.html
+++ b/index.html
@@ -201,6 +201,8 @@ <h2>
         </li>
         <li>[=manifest/theme_color=]
         </li>
+        <li>[=manifest/id=]
+        </li>
       </ul>
       <aside class="note">
         <p>
@@ -242,6 +244,7 @@ <h3>
                 "sizes": "128x128"
               }],
               "scope": "/",
+              "id": "superracer",
               "start_url": "/start.html",
               "display": "fullscreen",
               "orientation": "landscape",
@@ -808,6 +811,53 @@ <h3>
           </p>
         </section>
       </section>
+      <section>
+        <h3>
+          `id` member
+        </h3>
+        <p>
+          The [=manifest's=] <code><dfn data-export="" data-dfn-for=
+          "manifest">id</dfn></code> member is a <a>string</a> that
+          represents the <dfn>id</dfn> for the application.
+        </p>
+        <p>The |id| is used  by user agents uniquely identify the application
+          globally. When |id| is changed, it's recognized as a new app and
+          can be installed separately. When id is not changed, it can be used
+          to find installed application to perform manifest update.
+        </p>
+        <p>
+          The |id| can also be used by third party entities such as PWA stores
+          to uniquely identify and reference web apps
+        </p>
+        <p>
+          To <dfn>process the `id` member</dfn>, given [=object=]
+          |json:JSON|, [=ordered map=] |manifest:ordered map|.:
+        </p>
+        <ol class="algorithm">
+          <li>If the type of |json|["id"] is not [=string=], return.
+          </li>
+          <li>Let |id:URL| be the result of [=URL Parser|parsing=]
+          (<var>start URL's origin</var> + "/" + |json|["id"]).
+          </li>
+          <li>If |id| is failure, return.
+          </li>
+          <li>Otherwise, set |manifest|["id"] to |start URL|.
+          </li>
+        </ol>
+        <aside class="example">
+          <p>
+            For example, if the |start URL| is
+            <samp>https://example.com/resources/start</samp>,
+            and the value of [=manifest/id=] is <samp>"1"</samp>, then the
+            processed result is <samp>https://example.com/1</samp>. If the
+            value of [=manifest/id=] is <samp>""</samp>, then the processed
+            result is <samp>https://example.com/</samp>. If [=manifest/id=]
+            not specified, then the processed result is
+            <samp>https://example.com/start</samp>.
+
+          </p>
+        </aside>
+      </section>
       <section>
         <h3>
           `theme_color` member
@@ -1084,6 +1134,9 @@ <h3>
             <li>[=Process the `start_url` member=] passing |json|, |manifest|,
             |manifest URL|, and |document URL|.
             </li>
+            <li>[=Process the `id` member=] passing |json|, |manifest| and
+            |document URL|.
+            </li>
             <li>[=Process the `scope` member=] passing |json|, |manifest|, and
             |manifest URL|.
             </li>

From d3885d27637f82e83e8e4e1e94c7ef18d7aeb07a Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Thu, 15 Jul 2021 15:37:32 -0700
Subject: [PATCH 02/37] Describe how a manifest can be updated with id

---
 index.html | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/index.html b/index.html
index 6474703c4..5b6e66529 100644
--- a/index.html
+++ b/index.html
@@ -1262,6 +1262,13 @@ <h3>
             Updating the manifest
           </h3>
           <aside class="issue" data-number="446"></aside>
+          <p>
+            User agents MAY re-fetch and update a manifest for an installed web
+            application and apply updated manifest to its <a>application
+             context</a>. When a manifest is processed, user agents can
+            determine whether it's an installed application that can be updated
+            or a new application by comparing application's [=manifest/id=].
+          </p>
         </section>
       </section>
     </section>

From eb703811d3312dbdd00406b179c064921407231b Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Tue, 20 Jul 2021 13:28:40 -0700
Subject: [PATCH 03/37] address comments

---
 index.html | 57 +++++++++++++++++++++++++++++++++---------------------
 1 file changed, 35 insertions(+), 22 deletions(-)

diff --git a/index.html b/index.html
index 5b6e66529..973ed4464 100644
--- a/index.html
+++ b/index.html
@@ -201,7 +201,7 @@ <h2>
         </li>
         <li>[=manifest/theme_color=]
         </li>
-        <li>[=manifest/id=]
+        <li>[=manifest/identity=]
         </li>
       </ul>
       <aside class="note">
@@ -817,41 +817,51 @@ <h3>
         </h3>
         <p>
           The [=manifest's=] <code><dfn data-export="" data-dfn-for=
-          "manifest">id</dfn></code> member is a <a>string</a> that
-          represents the <dfn>id</dfn> for the application.
+          "manifest" id="identity">id</dfn></code> member is a <a>string</a> that
+          represents the <dfn>identity</dfn> for the application. 
+          The |identity| takes the form of a URL, which is same origin as the
+          start URL.
         </p>
-        <p>The |id| is used  by user agents uniquely identify the application
-          globally. When |id| is changed, it's recognized as a new app and
-          can be installed separately. When id is not changed, it can be used
-          to find installed application to perform manifest update.
+        <p>The |identity| is used by user agents to uniquely identify the
+        application universally. When the user agent sees a manifest with an
+        |identity| that does not correspond to an already-installed application,
+        it SHOULD treat that manifest as a description of a distinct
+        application, even if it is served from the same URL as that of another
+        application. When the user agent sees a manifest with an |identity|
+        matching that of an already-installed application, it SHOULD be used as
+        a signal that this manifest is a replacement for the already-installed
+        application's manifest, and not a distinct application, even if it is
+        served from a different URL than the one seen previously.
         </p>
         <p>
-          The |id| can also be used by third party entities such as PWA stores
-          to uniquely identify and reference web apps
+          The |identity| MAY be used by a service that collects lists of web
+          applications to uniquely identify applications.
         </p>
         <p>
           To <dfn>process the `id` member</dfn>, given [=object=]
-          |json:JSON|, [=ordered map=] |manifest:ordered map|.:
+          |json:JSON|, [=ordered map=] |manifest:ordered map|:
         </p>
         <ol class="algorithm">
+          <li>Set |manifest|["id"] to |manifest|["start_url"].
+          </li>
           <li>If the type of |json|["id"] is not [=string=], return.
           </li>
-          <li>Let |id:URL| be the result of [=URL Parser|parsing=]
-          (<var>start URL's origin</var> + "/" + |json|["id"]).
+          <li>Let |id:URL| be the result of [=URL Parser|parsing=] |json|["id"]
+          with |manifest|["start_ur"]'s origin as the base URL.
           </li>
-          <li>If |id| is failure, return.
+          <li>If |id| is not same origin as |manifest|["start_url"], return.
           </li>
-          <li>Otherwise, set |manifest|["id"] to |start URL|.
+          <li>If |id| is failure, return.
           </li>
         </ol>
         <aside class="example">
           <p>
             For example, if the |start URL| is
             <samp>https://example.com/resources/start</samp>,
-            and the value of [=manifest/id=] is <samp>"1"</samp>, then the
+            and the value of [=manifest/identity=] is <samp>"1"</samp>, then the
             processed result is <samp>https://example.com/1</samp>. If the
-            value of [=manifest/id=] is <samp>""</samp>, then the processed
-            result is <samp>https://example.com/</samp>. If [=manifest/id=]
+            value of [=manifest/identity=] is <samp>""</samp>, then the processed
+            result is <samp>https://example.com/</samp>. If [=manifest/identity=]
             not specified, then the processed result is
             <samp>https://example.com/start</samp>.
 
@@ -1263,11 +1273,14 @@ <h3>
           </h3>
           <aside class="issue" data-number="446"></aside>
           <p>
-            User agents MAY re-fetch and update a manifest for an installed web
-            application and apply updated manifest to its <a>application
-             context</a>. When a manifest is processed, user agents can
-            determine whether it's an installed application that can be updated
-            or a new application by comparing application's [=manifest/id=].
+            User agents MAY, at any time, re-fetch and update a manifest for an
+            installed web application and apply updated manifest to any current
+            and future <a>application contexts</a> associated with the
+            application. When the user agent fetches a manifest as part of an
+            update check, it MUST compare the fetched processed manifest's
+            [=manifest/identity=] with the identity of the app being updated. If
+            it is not equal, the user agent MUST NOT update the installed
+            application's manifest. 
           </p>
         </section>
       </section>

From fb3c3ffa7c4a0a110d372cfdaa0a80a105d4a44c Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Tue, 20 Jul 2021 14:15:19 -0700
Subject: [PATCH 04/37] Add example table

---
 index.html | 49 ++++++++++++++++++++++++++++++++++++++-----------
 1 file changed, 38 insertions(+), 11 deletions(-)

diff --git a/index.html b/index.html
index 973ed4464..e290c7c8b 100644
--- a/index.html
+++ b/index.html
@@ -855,17 +855,44 @@ <h3>
           </li>
         </ol>
         <aside class="example">
-          <p>
-            For example, if the |start URL| is
-            <samp>https://example.com/resources/start</samp>,
-            and the value of [=manifest/identity=] is <samp>"1"</samp>, then the
-            processed result is <samp>https://example.com/1</samp>. If the
-            value of [=manifest/identity=] is <samp>""</samp>, then the processed
-            result is <samp>https://example.com/</samp>. If [=manifest/identity=]
-            not specified, then the processed result is
-            <samp>https://example.com/start</samp>.
-
-          </p>
+          Below table shows some example cases of the |identity| processing algorithm.
+          <table class="data">
+            <tr>
+              <th>|json|["id"]</th>
+              <th>|manifest|["start_url"]</th>
+              <th>|manifest|["id"]</th>
+            </tr>
+            <tr>
+              <td><i>missing</i></td>
+              <td>https://example.com/resources/start</td>
+              <td>https://example.com/resources/start</td>
+            </tr>
+            <tr>
+              <td>""</td>
+              <td>https://example.com/resources/start</td>
+              <td>https://example.com/</td>
+            </tr>
+            <tr>
+              <td>/</td>
+              <td>https://example.com/resources/start</td>
+              <td>https://example.com/</td>
+            </tr>
+            <tr>
+              <td>foo</td>
+              <td>https://example.com/resources/start</td>
+              <td>https://example.com/foo</td>
+            </tr>
+            <tr>
+              <td>https://example.com/foo</td>
+              <td>https://example.com/resources/start</td>
+              <td>https://example.com/foo</td>
+            </tr>
+            <tr>
+              <td>https://anothersite.com/foo</td>
+              <td>https://example.com/resources/start</td>
+              <td>https://example.com/resources/start</td>
+            </tr>
+          </table>
         </aside>
       </section>
       <section>

From 997c58a6714ff905bcdaa28da63b68ecb0c0bf59 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Tue, 20 Jul 2021 14:19:08 -0700
Subject: [PATCH 05/37] re-order list

---
 index.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/index.html b/index.html
index e290c7c8b..7868b3303 100644
--- a/index.html
+++ b/index.html
@@ -181,6 +181,8 @@ <h2>
         </li>
         <li>[=manifest/icons=]
         </li>
+        <li>[=manifest/identity=]
+        </li>
         <li>[=manifest/lang=]
         </li>
         <li>[=manifest/name=]
@@ -201,8 +203,6 @@ <h2>
         </li>
         <li>[=manifest/theme_color=]
         </li>
-        <li>[=manifest/identity=]
-        </li>
       </ul>
       <aside class="note">
         <p>

From 4afda14ec5eb9b1927385a336baebb594638147b Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Tue, 20 Jul 2021 14:26:05 -0700
Subject: [PATCH 06/37] quote all data

---
 index.html | 32 ++++++++++++++++----------------
 1 file changed, 16 insertions(+), 16 deletions(-)

diff --git a/index.html b/index.html
index 3ad075a7e..07d005659 100644
--- a/index.html
+++ b/index.html
@@ -864,33 +864,33 @@ <h3>
             </tr>
             <tr>
               <td><i>missing</i></td>
-              <td>https://example.com/resources/start</td>
-              <td>https://example.com/resources/start</td>
+              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/resources/start"</td>
             </tr>
             <tr>
               <td>""</td>
-              <td>https://example.com/resources/start</td>
-              <td>https://example.com/</td>
+              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/"</td>
             </tr>
             <tr>
-              <td>/</td>
-              <td>https://example.com/resources/start</td>
-              <td>https://example.com/</td>
+              <td>"/"</td>
+              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/"</td>
             </tr>
             <tr>
-              <td>foo</td>
-              <td>https://example.com/resources/start</td>
-              <td>https://example.com/foo</td>
+              <td>"foo"</td>
+              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/foo"</td>
             </tr>
             <tr>
-              <td>https://example.com/foo</td>
-              <td>https://example.com/resources/start</td>
-              <td>https://example.com/foo</td>
+              <td>"https://example.com/foo"</td>
+              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/foo"</td>
             </tr>
             <tr>
-              <td>https://anothersite.com/foo</td>
-              <td>https://example.com/resources/start</td>
-              <td>https://example.com/resources/start</td>
+              <td>"https://anothersite.com/foo"</td>
+              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/resources/start"</td>
             </tr>
           </table>
         </aside>

From 1817348b6ada7edbceab21ba14c36d4979138cc2 Mon Sep 17 00:00:00 2001
From: Yajing Tang <phillis@google.com>
Date: Fri, 13 Aug 2021 14:39:08 -0700
Subject: [PATCH 07/37] Update index.html
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index 07d005659..4b27d84a8 100644
--- a/index.html
+++ b/index.html
@@ -863,7 +863,7 @@ <h3>
               <th>|manifest|["id"]</th>
             </tr>
             <tr>
-              <td><i>missing</i></td>
+              <td><I>undefined</i></td>
               <td>"https://example.com/resources/start"</td>
               <td>"https://example.com/resources/start"</td>
             </tr>

From f743f90df04103b6d4f5b3f25e870711599dfac7 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Fri, 13 Aug 2021 14:50:53 -0700
Subject: [PATCH 08/37] update algorithm

---
 index.html | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/index.html b/index.html
index 07d005659..7d2c484a9 100644
--- a/index.html
+++ b/index.html
@@ -849,10 +849,11 @@ <h3>
           <li>Let |id:URL| be the result of [=URL Parser|parsing=] |json|["id"]
           with |manifest|["start_ur"]'s origin as the base URL.
           </li>
-          <li>If |id| is not same origin as |manifest|["start_url"], return.
-          </li>
           <li>If |id| is failure, return.
           </li>
+          <li>If |id| is not same origin as |manifest|["start_url"], return.
+          </li>
+          <li>Otherwise, set |manifest|["id"] to |id|.</li>
         </ol>
         <aside class="example">
           Below table shows some example cases of the |identity| processing algorithm.
@@ -863,7 +864,7 @@ <h3>
               <th>|manifest|["id"]</th>
             </tr>
             <tr>
-              <td><i>missing</i></td>
+              <td><i>undefined</i></td>
               <td>"https://example.com/resources/start"</td>
               <td>"https://example.com/resources/start"</td>
             </tr>

From 3659d388b7ba39f7a2a3e3867829c04321d3d206 Mon Sep 17 00:00:00 2001
From: Yajing Tang <phillis@google.com>
Date: Fri, 13 Aug 2021 14:52:06 -0700
Subject: [PATCH 09/37] Update index.html
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
---
 index.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/index.html b/index.html
index 4b27d84a8..a601543b1 100644
--- a/index.html
+++ b/index.html
@@ -833,8 +833,8 @@ <h3>
         application's manifest, and not a distinct application, even if it is
         served from a different URL than the one seen previously.
         </p>
-        <p>
-          The |identity| MAY be used by a service that collects lists of web
+        <p class="note">
+          The |identity| can be used by a service that collects lists of web
           applications to uniquely identify applications.
         </p>
         <p>

From 52186068fad48d03bac4a79da36984f1508698be Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Fri, 13 Aug 2021 15:23:49 -0700
Subject: [PATCH 10/37] update manifest update description

---
 .vscode/c_cpp_properties.json | 16 ++++++++++++++++
 index.html                    | 27 +++++++++++++++++++--------
 2 files changed, 35 insertions(+), 8 deletions(-)
 create mode 100644 .vscode/c_cpp_properties.json

diff --git a/.vscode/c_cpp_properties.json b/.vscode/c_cpp_properties.json
new file mode 100644
index 000000000..9e6febff5
--- /dev/null
+++ b/.vscode/c_cpp_properties.json
@@ -0,0 +1,16 @@
+{
+    "configurations": [
+        {
+            "name": "Linux",
+            "includePath": [
+                "${workspaceFolder}/**"
+            ],
+            "defines": [],
+            "compilerPath": "/usr/bin/clang",
+            "cStandard": "c17",
+            "cppStandard": "c++14",
+            "intelliSenseMode": "linux-clang-x64"
+        }
+    ],
+    "version": 4
+}
\ No newline at end of file
diff --git a/index.html b/index.html
index dc6638f46..ed6512acb 100644
--- a/index.html
+++ b/index.html
@@ -1302,15 +1302,26 @@ <h3>
           </h3>
           <aside class="issue" data-number="446"></aside>
           <p>
-            User agents MAY, at any time, re-fetch and update a manifest for an
-            installed web application and apply updated manifest to any current
-            and future <a>application contexts</a> associated with the
-            application. When the user agent fetches a manifest as part of an
-            update check, it MUST compare the fetched processed manifest's
-            [=manifest/identity=] with the identity of the app being updated. If
-            it is not equal, the user agent MUST NOT update the installed
-            application's manifest. 
+            User agents SHOULD re-fetch and process the manifest on every page
+            load. When the processing is successful, user agents MAY apply updated
+            manifest to any current and future <a>application contexts</a>
+            associated with the application. When the user agent fetches a
+            manifest as part of an update check, it MUST compare the fetched
+            processed manifest's [=manifest/identity=] with the identity of the
+            app being updated. If it is not equal, the user agent MUST NOT
+            update the installed application's manifest. 
           </p>
+          <section>
+            <h3>
+              Privacy and security consideration
+            </h3>
+            <p>
+              User agents SHOULD be careful about performing update to manifest
+              fields like app name and icons. Users should be fully aware if
+              such fields are to be be updated to avoid web applications being
+              installed and silently changed to appear as a different one.
+            </p>
+          </section>
         </section>
       </section>
     </section>

From 490a94837b2837ea3af77924b601c1487faa86b0 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Mon, 16 Aug 2021 12:49:38 -0700
Subject: [PATCH 11/37] Clarify id equivalence

---
 index.html | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/index.html b/index.html
index ed6512acb..ad9fe5a03 100644
--- a/index.html
+++ b/index.html
@@ -833,6 +833,9 @@ <h3>
         application's manifest, and not a distinct application, even if it is
         served from a different URL than the one seen previously.
         </p>
+        <p>
+        The |identity| is considered matching when the processed |manifest|["id"] [=url/equal|equals=].
+        </p>
         <p class="note">
           The |identity| can be used by a service that collects lists of web
           applications to uniquely identify applications.

From 4e05505db766de97b9cf24f1eb229511ca24a6ac Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Mon, 16 Aug 2021 12:51:39 -0700
Subject: [PATCH 12/37] refence url/equal concept in manifest update

---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index ad9fe5a03..bbfaecf64 100644
--- a/index.html
+++ b/index.html
@@ -1309,7 +1309,7 @@ <h3>
             load. When the processing is successful, user agents MAY apply updated
             manifest to any current and future <a>application contexts</a>
             associated with the application. When the user agent fetches a
-            manifest as part of an update check, it MUST compare the fetched
+            manifest as part of an update check, it MUST [=url/equal|compare=] the fetched
             processed manifest's [=manifest/identity=] with the identity of the
             app being updated. If it is not equal, the user agent MUST NOT
             update the installed application's manifest. 

From e6c7461bc5274de2853ca516c4211f67c8fdfb49 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Tue, 17 Aug 2021 12:00:21 -0700
Subject: [PATCH 13/37] remove vscode file

---
 .vscode/c_cpp_properties.json | 16 ----------------
 1 file changed, 16 deletions(-)
 delete mode 100644 .vscode/c_cpp_properties.json

diff --git a/.vscode/c_cpp_properties.json b/.vscode/c_cpp_properties.json
deleted file mode 100644
index 9e6febff5..000000000
--- a/.vscode/c_cpp_properties.json
+++ /dev/null
@@ -1,16 +0,0 @@
-{
-    "configurations": [
-        {
-            "name": "Linux",
-            "includePath": [
-                "${workspaceFolder}/**"
-            ],
-            "defines": [],
-            "compilerPath": "/usr/bin/clang",
-            "cStandard": "c17",
-            "cppStandard": "c++14",
-            "intelliSenseMode": "linux-clang-x64"
-        }
-    ],
-    "version": 4
-}
\ No newline at end of file

From 73515ef9da2959d2a0260256637a18fd7435c98f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Tue, 21 Sep 2021 13:54:25 +1000
Subject: [PATCH 14/37] Update index.html

---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index 28081a52a..8a04199c2 100644
--- a/index.html
+++ b/index.html
@@ -853,7 +853,7 @@ <h3>
           <li>If the type of |json|["id"] is not [=string=], return.
           </li>
           <li>Let |id:URL| be the result of [=URL Parser|parsing=] |json|["id"]
-          with |manifest|["start_ur"]'s origin as the base URL.
+          with |manifest|["start_url"]'s origin as the base URL.
           </li>
           <li>If |id| is failure, return.
           </li>

From b73baed232534ea817a6c75f971968ad512ec7ae Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Tue, 21 Sep 2021 13:54:38 +1000
Subject: [PATCH 15/37] Update index.html

---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index 8a04199c2..697e81200 100644
--- a/index.html
+++ b/index.html
@@ -822,7 +822,7 @@ <h3>
           The [=manifest's=] <code><dfn data-export="" data-dfn-for=
           "manifest" id="identity">id</dfn></code> member is a <a>string</a> that
           represents the <dfn>identity</dfn> for the application. 
-          The |identity| takes the form of a URL, which is same origin as the
+          The [=identity=] takes the form of a URL, which is same origin as the
           start URL.
         </p>
         <p>The |identity| is used by user agents to uniquely identify the

From 579667ece869c851f7b40358a6ea1ce2604d1780 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Tue, 21 Sep 2021 13:59:39 +1000
Subject: [PATCH 16/37] Update index.html

---
 index.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/index.html b/index.html
index 697e81200..57b4425db 100644
--- a/index.html
+++ b/index.html
@@ -825,12 +825,12 @@ <h3>
           The [=identity=] takes the form of a URL, which is same origin as the
           start URL.
         </p>
-        <p>The |identity| is used by user agents to uniquely identify the
+        <p>The [=identity=] is used by user agents to uniquely identify the
         application universally. When the user agent sees a manifest with an
-        |identity| that does not correspond to an already-installed application,
+        [=identity=] that does not correspond to an already-installed application,
         it SHOULD treat that manifest as a description of a distinct
         application, even if it is served from the same URL as that of another
-        application. When the user agent sees a manifest with an |identity|
+        application. When the user agent sees a manifest with an [=identity=]
         matching that of an already-installed application, it SHOULD be used as
         a signal that this manifest is a replacement for the already-installed
         application's manifest, and not a distinct application, even if it is

From 23c5ac7624de7b8fc39023fdadcfb43f7e567911 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Tue, 21 Sep 2021 15:00:18 +1000
Subject: [PATCH 17/37] Update index.html

---
 index.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/index.html b/index.html
index 57b4425db..0f3914ade 100644
--- a/index.html
+++ b/index.html
@@ -837,10 +837,10 @@ <h3>
         served from a different URL than the one seen previously.
         </p>
         <p>
-        The |identity| is considered matching when the processed |manifest|["id"] [=url/equal|equals=].
+        The [=identity=] is considered matching when the processed |manifest|["id"] [=url/equal|equals=].
         </p>
         <p class="note">
-          The |identity| can be used by a service that collects lists of web
+          The [=identity=] can be used by a service that collects lists of web
           applications to uniquely identify applications.
         </p>
         <p>

From 6c29909d8f831cffa817b260f6f593d2a963ec7d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Tue, 21 Sep 2021 15:00:53 +1000
Subject: [PATCH 18/37] Update index.html

---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index 0f3914ade..c7e7d8caa 100644
--- a/index.html
+++ b/index.html
@@ -862,7 +862,7 @@ <h3>
           <li>Otherwise, set |manifest|["id"] to |id|.</li>
         </ol>
         <aside class="example">
-          Below table shows some example cases of the |identity| processing algorithm.
+          <p>Below table shows some example cases of the |identity| processing algorithm.</p>
           <table class="data">
             <tr>
               <th>|json|["id"]</th>

From d29435f2b9566775efffc4b4021ddcc13a080cea Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Tue, 21 Sep 2021 15:33:02 +1000
Subject: [PATCH 19/37] Update index.html

---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index c7e7d8caa..5228ba3bb 100644
--- a/index.html
+++ b/index.html
@@ -857,7 +857,7 @@ <h3>
           </li>
           <li>If |id| is failure, return.
           </li>
-          <li>If |id| is not same origin as |manifest|["start_url"], return.
+          <li>If |id| is not [=same origin=] as |manifest|["start_url"], return.
           </li>
           <li>Otherwise, set |manifest|["id"] to |id|.</li>
         </ol>

From e1ab8a3c7ec1a95d12cc05cf6d0045c2ddfed5a9 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Tue, 21 Sep 2021 16:00:34 -0700
Subject: [PATCH 20/37] address some comments

---
 index.html | 40 ++++++++++++++++++++++++----------------
 1 file changed, 24 insertions(+), 16 deletions(-)

diff --git a/index.html b/index.html
index 5228ba3bb..9029b0626 100644
--- a/index.html
+++ b/index.html
@@ -871,33 +871,38 @@ <h3>
             </tr>
             <tr>
               <td><i>undefined</i></td>
-              <td>"https://example.com/resources/start"</td>
-              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/my-app/start"</td>
+              <td>"https://example.com/my-app/start"</td>
             </tr>
             <tr>
               <td>""</td>
-              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/my-app/start"</td>
               <td>"https://example.com/"</td>
             </tr>
             <tr>
               <td>"/"</td>
-              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/my-app/start"</td>
               <td>"https://example.com/"</td>
             </tr>
             <tr>
               <td>"foo"</td>
-              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/my-app/start"</td>
               <td>"https://example.com/foo"</td>
             </tr>
             <tr>
+              <td>"./foo"</td>
+              <td>"https://example.com/my-app/start"</td>
               <td>"https://example.com/foo"</td>
-              <td>"https://example.com/resources/start"</td>
+            </tr>
+            <tr>
+              <td>"https://example.com/foo"</td>
+              <td>"https://example.com/my-app/start"</td>
               <td>"https://example.com/foo"</td>
             </tr>
             <tr>
               <td>"https://anothersite.com/foo"</td>
-              <td>"https://example.com/resources/start"</td>
-              <td>"https://example.com/resources/start"</td>
+              <td>"https://example.com/my-app/start"</td>
+              <td>"https://example.com/my-app/start"</td>
             </tr>
           </table>
         </aside>
@@ -1317,14 +1322,17 @@ <h3>
           </h3>
           <aside class="issue" data-number="446"></aside>
           <p>
-            User agents SHOULD re-fetch and process the manifest on every page
-            load. When the processing is successful, user agents MAY apply updated
-            manifest to any current and future <a>application contexts</a>
-            associated with the application. When the user agent fetches a
-            manifest as part of an update check, it MUST [=url/equal|compare=] the fetched
-            processed manifest's [=manifest/identity=] with the identity of the
-            app being updated. If it is not equal, the user agent MUST NOT
-            update the installed application's manifest. 
+            The manifest is fetched and processed on every page load. When the
+            processing is successful, user agents MAY apply updated manifest to
+            any current and future <a>application contexts</a>
+            associated with the application. 
+          </p>
+          <p>
+            When the user agent fetches a manifest as part of an update check,
+            it MUST [=url/equal|compare=] the fetched processed manifest's
+            [=manifest/identity=] with the identity of the app being updated. If
+            it is not equal, the user agent MUST NOT update the installed
+            application's manifest. 
           </p>
           <section>
             <h3>

From d689b76b360f3a010ab7fb67be5cf9ee2760ea23 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Tue, 21 Sep 2021 16:53:24 -0700
Subject: [PATCH 21/37] add more examples

---
 index.html | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/index.html b/index.html
index 9029b0626..0ce1b3db0 100644
--- a/index.html
+++ b/index.html
@@ -843,6 +843,11 @@ <h3>
           The [=identity=] can be used by a service that collects lists of web
           applications to uniquely identify applications.
         </p>
+        <p class="note">
+          The [=identity=] is processed like a URL but it doesn't point to a
+          resource that can be navigated to, so it's not required to be
+          [=URL/within scope=].
+        </p>
         <p>
           To <dfn>process the `id` member</dfn>, given [=object=]
           |json:JSON|, [=ordered map=] |manifest:ordered map|:
@@ -904,7 +909,23 @@ <h3>
               <td>"https://example.com/my-app/start"</td>
               <td>"https://example.com/my-app/start"</td>
             </tr>
+            <tr>
+              <td>"%6D"</td>
+              <td>"https://example.com/my-app/start"</td>
+              <td>"https://example.com/m"</td>
+            </tr>
+            <tr>
+              <td>"😀"</td>
+              <td>"https://example.com/my-app/start"</td>
+              <td>"https://example.com/%F0%9F%98%80"</td>
+            </tr>
           </table>
+          <p>Since id is resolved against start_url's origin, providing id as
+          "../foo", "foo", "/foo", "./foo" all resolves to the same thing. It is
+          RECOMMENDED to always specify with a leading "/" to be explicit that
+          the id is a root-relative url path. Also the same url
+          encoding/decoding rules are applied to the id processing algorithm.
+          </p>
         </aside>
       </section>
       <section>

From 66a166100f135a376a76d7f90e63edebdf6885e6 Mon Sep 17 00:00:00 2001
From: Yajing Tang <phillis@google.com>
Date: Wed, 22 Sep 2021 17:07:37 -0700
Subject: [PATCH 22/37] Update index.html
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index 0ce1b3db0..bc40bea39 100644
--- a/index.html
+++ b/index.html
@@ -922,7 +922,7 @@ <h3>
           </table>
           <p>Since id is resolved against start_url's origin, providing id as
           "../foo", "foo", "/foo", "./foo" all resolves to the same thing. It is
-          RECOMMENDED to always specify with a leading "/" to be explicit that
+          As such, best practice is to use a leading "/" to be explicit that
           the id is a root-relative url path. Also the same url
           encoding/decoding rules are applied to the id processing algorithm.
           </p>

From a006387880294055ad1b3a657d43099d139615a6 Mon Sep 17 00:00:00 2001
From: Yajing Tang <phillis@google.com>
Date: Wed, 22 Sep 2021 17:07:47 -0700
Subject: [PATCH 23/37] Update index.html
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
---
 index.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/index.html b/index.html
index bc40bea39..ccec27a1d 100644
--- a/index.html
+++ b/index.html
@@ -923,8 +923,8 @@ <h3>
           <p>Since id is resolved against start_url's origin, providing id as
           "../foo", "foo", "/foo", "./foo" all resolves to the same thing. It is
           As such, best practice is to use a leading "/" to be explicit that
-          the id is a root-relative url path. Also the same url
-          encoding/decoding rules are applied to the id processing algorithm.
+          the id is a root-relative url path. Also, standard
+          encoding/decoding rules are apply to the id processing algorithm, as per the [[[URL]]].
           </p>
         </aside>
       </section>

From 171575cd81afc82baf88e93168479286e4a22b9c Mon Sep 17 00:00:00 2001
From: Yajing Tang <phillis@google.com>
Date: Wed, 22 Sep 2021 17:08:28 -0700
Subject: [PATCH 24/37] Update index.html
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index ccec27a1d..52d478f8e 100644
--- a/index.html
+++ b/index.html
@@ -921,7 +921,7 @@ <h3>
             </tr>
           </table>
           <p>Since id is resolved against start_url's origin, providing id as
-          "../foo", "foo", "/foo", "./foo" all resolves to the same thing. It is
+          "../foo", "foo", "/foo", "./foo" all resolves to the same [=identifier=]. It is
           As such, best practice is to use a leading "/" to be explicit that
           the id is a root-relative url path. Also, standard
           encoding/decoding rules are apply to the id processing algorithm, as per the [[[URL]]].

From d4a7ac156478043982e8409815d2d6e1217ee85d Mon Sep 17 00:00:00 2001
From: Yajing Tang <phillis@google.com>
Date: Wed, 22 Sep 2021 17:08:49 -0700
Subject: [PATCH 25/37] Update index.html
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index 52d478f8e..a6a78eeea 100644
--- a/index.html
+++ b/index.html
@@ -920,7 +920,7 @@ <h3>
               <td>"https://example.com/%F0%9F%98%80"</td>
             </tr>
           </table>
-          <p>Since id is resolved against start_url's origin, providing id as
+          <p>Since [=manifest/id=] is resolved against [=manifest/start_url=]'s [=URL/origin=], providing
           "../foo", "foo", "/foo", "./foo" all resolves to the same [=identifier=]. It is
           As such, best practice is to use a leading "/" to be explicit that
           the id is a root-relative url path. Also, standard

From a222c0d9ea308a0e86d97269aeb9d66df4e4792f Mon Sep 17 00:00:00 2001
From: Yajing Tang <phillis@google.com>
Date: Wed, 22 Sep 2021 17:09:22 -0700
Subject: [PATCH 26/37] Update index.html
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
---
 index.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/index.html b/index.html
index a6a78eeea..db9335a4e 100644
--- a/index.html
+++ b/index.html
@@ -857,8 +857,8 @@ <h3>
           </li>
           <li>If the type of |json|["id"] is not [=string=], return.
           </li>
-          <li>Let |id:URL| be the result of [=URL Parser|parsing=] |json|["id"]
-          with |manifest|["start_url"]'s origin as the base URL.
+          <li>Let |base origin| be |manifest|["start_url"]'s [=url/origin=].</li> 
+          <li>Let |id:URL| be the result of [=URL Parser|parsing=] |json|["id"] with  |base origin| as the base URL.
           </li>
           <li>If |id| is failure, return.
           </li>

From 571f64c0502679d7babc743f0b5c601ab6582180 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Wed, 22 Sep 2021 17:44:57 -0700
Subject: [PATCH 27/37] update example for empty string

---
 index.html | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/index.html b/index.html
index db9335a4e..d3dc94dd3 100644
--- a/index.html
+++ b/index.html
@@ -873,6 +873,7 @@ <h3>
               <th>|json|["id"]</th>
               <th>|manifest|["start_url"]</th>
               <th>|manifest|["id"]</th>
+              <th>Explaination</th>
             </tr>
             <tr>
               <td><i>undefined</i></td>
@@ -882,7 +883,7 @@ <h3>
             <tr>
               <td>""</td>
               <td>"https://example.com/my-app/start"</td>
-              <td>"https://example.com/"</td>
+              <td>"https://example.com/my-app/start"</td>
             </tr>
             <tr>
               <td>"/"</td>

From ee923ad544f58baeb1efa582cd52219d06f529ec Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Wed, 22 Sep 2021 17:49:54 -0700
Subject: [PATCH 28/37] update processing algorithm

---
 index.html | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/index.html b/index.html
index d3dc94dd3..86188e778 100644
--- a/index.html
+++ b/index.html
@@ -857,6 +857,8 @@ <h3>
           </li>
           <li>If the type of |json|["id"] is not [=string=], return.
           </li>
+          <li>If |json|["id"] is the empty string, return.
+          </li>
           <li>Let |base origin| be |manifest|["start_url"]'s [=url/origin=].</li> 
           <li>Let |id:URL| be the result of [=URL Parser|parsing=] |json|["id"] with  |base origin| as the base URL.
           </li>

From f9ca5a095dea4c28b53b901601c32895ac867ef8 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Thu, 23 Sep 2021 14:33:53 -0700
Subject: [PATCH 29/37] remove ascii code example

---
 index.html | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/index.html b/index.html
index 86188e778..99b245ea1 100644
--- a/index.html
+++ b/index.html
@@ -912,11 +912,6 @@ <h3>
               <td>"https://example.com/my-app/start"</td>
               <td>"https://example.com/my-app/start"</td>
             </tr>
-            <tr>
-              <td>"%6D"</td>
-              <td>"https://example.com/my-app/start"</td>
-              <td>"https://example.com/m"</td>
-            </tr>
             <tr>
               <td>"😀"</td>
               <td>"https://example.com/my-app/start"</td>

From 3a97b9e7443c59ba2d25686abb15c21ab34d9789 Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Fri, 24 Sep 2021 12:19:37 -0700
Subject: [PATCH 30/37] Re-phrase identity equality section

---
 index.html | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/index.html b/index.html
index 99b245ea1..056ea4762 100644
--- a/index.html
+++ b/index.html
@@ -827,17 +827,15 @@ <h3>
         </p>
         <p>The [=identity=] is used by user agents to uniquely identify the
         application universally. When the user agent sees a manifest with an
-        [=identity=] that does not correspond to an already-installed application,
-        it SHOULD treat that manifest as a description of a distinct
-        application, even if it is served from the same URL as that of another
-        application. When the user agent sees a manifest with an [=identity=]
-        matching that of an already-installed application, it SHOULD be used as
-        a signal that this manifest is a replacement for the already-installed
-        application's manifest, and not a distinct application, even if it is
-        served from a different URL than the one seen previously.
-        </p>
-        <p>
-        The [=identity=] is considered matching when the processed |manifest|["id"] [=url/equal|equals=].
+        [=identity=] that does not correspond to an already-installed
+        application, it SHOULD treat that manifest as a description of a
+        distinct application, even if it is served from the same URL as that of
+        another application. When the user agent sees a manifest where
+        |manifest|["id"] [=url/equal|equals=] the [=identity=] of an
+        already-installed application, it SHOULD be used as a signal that this
+        manifest is a replacement for the already-installed application's
+        manifest, and not a distinct application, even if it is served from a
+        different URL than the one seen previously.
         </p>
         <p class="note">
           The [=identity=] can be used by a service that collects lists of web

From bb5b1170322114473b99eebe6732293d604783fc Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Mon, 27 Sep 2021 14:06:47 -0700
Subject: [PATCH 31/37] remove fragment from default and processed id

---
 index.html | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/index.html b/index.html
index 056ea4762..5a7a59904 100644
--- a/index.html
+++ b/index.html
@@ -851,7 +851,7 @@ <h3>
           |json:JSON|, [=ordered map=] |manifest:ordered map|:
         </p>
         <ol class="algorithm">
-          <li>Set |manifest|["id"] to |manifest|["start_url"].
+          <li>Set |manifest|["id"] to |manifest|["start_url"] with [=URL/fragment=] removed.
           </li>
           <li>If the type of |json|["id"] is not [=string=], return.
           </li>
@@ -864,7 +864,8 @@ <h3>
           </li>
           <li>If |id| is not [=same origin=] as |manifest|["start_url"], return.
           </li>
-          <li>Otherwise, set |manifest|["id"] to |id|.</li>
+          <li>Remove [=URL/fragment=] from |id|.</li>
+          <li>Set |manifest|["id"] to |id|.</li>
         </ol>
         <aside class="example">
           <p>Below table shows some example cases of the |identity| processing algorithm.</p>
@@ -873,13 +874,17 @@ <h3>
               <th>|json|["id"]</th>
               <th>|manifest|["start_url"]</th>
               <th>|manifest|["id"]</th>
-              <th>Explaination</th>
             </tr>
             <tr>
               <td><i>undefined</i></td>
               <td>"https://example.com/my-app/start"</td>
               <td>"https://example.com/my-app/start"</td>
             </tr>
+            <tr>
+              <td><i>undefined</i></td>
+              <td>"https://example.com/my-app/#here"</td>
+              <td>"https://example.com/my-app/"</td>
+            </tr>
             <tr>
               <td>""</td>
               <td>"https://example.com/my-app/start"</td>

From 3a70f207b7f31cbe14143a0d07e6bef6fb16960a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Wed, 29 Sep 2021 16:01:06 +1000
Subject: [PATCH 32/37] Apply suggestions from code review

---
 index.html | 1 -
 1 file changed, 1 deletion(-)

diff --git a/index.html b/index.html
index 5a7a59904..32f8fc4d9 100644
--- a/index.html
+++ b/index.html
@@ -1342,7 +1342,6 @@ <h3 id="applying">
           <h3>
             Updating the manifest
           </h3>
-          <aside class="issue" data-number="446"></aside>
           <p>
             The manifest is fetched and processed on every page load. When the
             processing is successful, user agents MAY apply updated manifest to

From f9d4edbbab1091cf8a45bf0864a3ed0deb021dd6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Wed, 29 Sep 2021 17:04:23 +1000
Subject: [PATCH 33/37] Update index.html

---
 index.html | 1 +
 1 file changed, 1 insertion(+)

diff --git a/index.html b/index.html
index 32f8fc4d9..5a7a59904 100644
--- a/index.html
+++ b/index.html
@@ -1342,6 +1342,7 @@ <h3 id="applying">
           <h3>
             Updating the manifest
           </h3>
+          <aside class="issue" data-number="446"></aside>
           <p>
             The manifest is fetched and processed on every page load. When the
             processing is successful, user agents MAY apply updated manifest to

From f89eda6520f90c45b7d42c0d81b06f2db165b5bb Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Wed, 29 Sep 2021 13:33:28 -0700
Subject: [PATCH 34/37] remove manifest update

---
 index.html | 25 -------------------------
 1 file changed, 25 deletions(-)

diff --git a/index.html b/index.html
index 5a7a59904..f38c5397a 100644
--- a/index.html
+++ b/index.html
@@ -1343,32 +1343,7 @@ <h3>
             Updating the manifest
           </h3>
           <aside class="issue" data-number="446"></aside>
-          <p>
-            The manifest is fetched and processed on every page load. When the
-            processing is successful, user agents MAY apply updated manifest to
-            any current and future <a>application contexts</a>
-            associated with the application. 
-          </p>
-          <p>
-            When the user agent fetches a manifest as part of an update check,
-            it MUST [=url/equal|compare=] the fetched processed manifest's
-            [=manifest/identity=] with the identity of the app being updated. If
-            it is not equal, the user agent MUST NOT update the installed
-            application's manifest. 
-          </p>
-          <section>
-            <h3>
-              Privacy and security consideration
-            </h3>
-            <p>
-              User agents SHOULD be careful about performing update to manifest
-              fields like app name and icons. Users should be fully aware if
-              such fields are to be be updated to avoid web applications being
-              installed and silently changed to appear as a different one.
-            </p>
-          </section>
         </section>
-      </section>
     </section>
     <section>
       <h2>

From 2fc8a9e795c1b7b559982b983219fe4879e3f1fd Mon Sep 17 00:00:00 2001
From: Phillis Tang <phillis@chromium.org>
Date: Wed, 29 Sep 2021 13:41:27 -0700
Subject: [PATCH 35/37] fix

---
 index.html | 1 +
 1 file changed, 1 insertion(+)

diff --git a/index.html b/index.html
index f38c5397a..f39110116 100644
--- a/index.html
+++ b/index.html
@@ -1344,6 +1344,7 @@ <h3>
           </h3>
           <aside class="issue" data-number="446"></aside>
         </section>
+      </section>
     </section>
     <section>
       <h2>

From f0817f0d9b2eacef261920ae3feae63c658039de Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Wed, 6 Oct 2021 10:28:46 +1100
Subject: [PATCH 36/37] Update index.html

---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index f39110116..17aa04a2b 100644
--- a/index.html
+++ b/index.html
@@ -851,7 +851,7 @@ <h3>
           |json:JSON|, [=ordered map=] |manifest:ordered map|:
         </p>
         <ol class="algorithm">
-          <li>Set |manifest|["id"] to |manifest|["start_url"] with [=URL/fragment=] removed.
+          <li>Set |manifest|["id"] to |manifest|["start_url"].
           </li>
           <li>If the type of |json|["id"] is not [=string=], return.
           </li>

From e753dc5d4f62454338889678a776cef1b0a80d9a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marcos=20C=C3=A1ceres?= <marcos@marcosc.com>
Date: Wed, 6 Oct 2021 10:29:19 +1100
Subject: [PATCH 37/37] Apply suggestions from code review

---
 index.html | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/index.html b/index.html
index 17aa04a2b..ae5930b67 100644
--- a/index.html
+++ b/index.html
@@ -864,7 +864,6 @@ <h3>
           </li>
           <li>If |id| is not [=same origin=] as |manifest|["start_url"], return.
           </li>
-          <li>Remove [=URL/fragment=] from |id|.</li>
           <li>Set |manifest|["id"] to |id|.</li>
         </ol>
         <aside class="example">
@@ -883,7 +882,7 @@ <h3>
             <tr>
               <td><i>undefined</i></td>
               <td>"https://example.com/my-app/#here"</td>
-              <td>"https://example.com/my-app/"</td>
+              <td>"https://example.com/my-app/#here"</td>
             </tr>
             <tr>
               <td>""</td>