Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix updating the session history to activate new Documents #6199

Merged
merged 9 commits into from
Dec 11, 2020
Merged

Fix updating the session history to activate new Documents #6199

Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
45a1c98
Fix updating the session history
domfarolino Dec 6, 2020
8ab810a
Clean my vim scoll helpers
domfarolino Dec 6, 2020
c5756b3
Likely fix build
domfarolino Dec 6, 2020
abf1c9d
Reflect the latest proposals
domfarolino Dec 11, 2020
7ddf9a3
Correct some minor mistakes
domfarolino Dec 11, 2020
55f83a7
Fix most things
domfarolino Dec 11, 2020
342aefc
Use navigation-params-hh instead of sameEntryTraversal flag
domfarolino Dec 11, 2020
6a3a6d6
Nits
domenic Dec 11, 2020
6608d77
Change the assertion while we're there
domenic Dec 11, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
107 changes: 72 additions & 35 deletions source
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85127,16 +85127,13 @@ new PaymentRequest(…); // Allowed to use

<p>Some of the sections below, to which the above algorithm defers in certain cases, require the
user agent to <dfn>update the session history with the new page</dfn>, given some <span>navigation
params</span> <var>navigationParams</var>. When a user agent is required to do this, it must
<span>queue a global task</span> on the <span>networking task source</span>, given the
<span>relevant global object</span> of the <code>Document</code> object of the <span>current
entry</span> (not the new one), to run the following steps:</p>
params</span> <var>navigationParams</var> and a <code>Document</code> <var>newDocument</var>. When
a user agent is required to do this, it must <span>queue a global task</span> on the
<span>networking task source</span>, given the <span>relevant global object</span> of the
<code>Document</code> object of the <span>current entry</span> (not the new one), to run the
following steps:</p>

<ol>
<li><p>Let <var>newDocument</var> be <var>navigationParams</var>'s <span
data-x="navigation-params-browsing-context">browsing context</span>'s <span>active
document</span>.</p></li>

<li>
<p><span data-x="unload a document">Unload</span> the <span
data-x="she-document">document</span> of the <span>current entry</span>.</p>
Expand All @@ -85152,38 +85149,76 @@ new PaymentRequest(&hellip;); // Allowed to use
<li>
<dl>
<dt>If <var>navigationParams</var>'s <span data-x="navigation-params-hh">history
handling</span> is "<code data-x="hh-reload">reload</code>" or "<code
data-x="hh-entry-update">entry update</code>"</dt>
handling</span> is "<code data-x="hh-entry-update">entry update</code>" or "<code
data-x="hh-reload">reload</code>"</dt>

<dd>
<ol>
<li><p>Replace the <span data-x="she-document">document</span> of the <span>current
entry</span>, and any other entries that reference the same <span
entry</span>, and of any other entries that reference the same <span
data-x="she-document">document</span> as that entry, with <var>newDocument</var>.</p></li>

<li><p><span>Traverse the history</span> to the <span>current entry</span>.</p></li>
<li><p><span>Traverse the history</span> to the <span>current entry</span> with <var
data-x="traverse-history-hh">historyHandling</var> set to <var>navigationParams</var>'s <span
data-x="navigation-params-hh">history handling</span>.</p></li>
</ol>
</dd>

<dt>If the navigation was initiated with a <span>URL</span> that <span
data-x="concept-url-equals">equals</span> <var>newDocument</var>'s <span
data-x="concept-document-url">URL</span></dt>
<dt>Otherwise, if the navigation was initiated with a <span>URL</span> that <span
data-x="concept-url-equals">equals</span> <var>navigationParams</var>'s <span
data-x="navigation-params-browsing-context">browsing context</span>'s <span>active
domfarolino marked this conversation as resolved.
Show resolved Hide resolved
document</span>'s <span data-x="concept-document-url">URL</span></dt>

<dd>
<ol>
domfarolino marked this conversation as resolved.
Show resolved Hide resolved
<li><p>Replace the <span>current entry</span> with a new <span>session history entry</span>
whose <span data-x="she-url">URL</span> is <var>newDocument</var>'s <span
data-x="concept-document-url">URL</span> and <span data-x="she-document">document</span> is
<var>newDocument</var>.</p></li>
<li>
<p>Let <var>newEntry</var> be a new <span>session history entry</span> whose <span
data-x="she-url">URL</span> is <var>newDocument</var>'s <span
data-x="concept-document-url">URL</span> and <span data-x="she-document">document</span> is
<var>newDocument</var>.</p>

<p class="XXX">Some browsers copy over the serialized state of <span>current entry</span>,
but this is inconsistent. See <a href="https://github.com/whatwg/html/issues/6213">issue
#6213</a> for more discussion on this.</p>
</li>

<li><p>Insert <var>newEntry</var> after the <span>current entry</span> in
<var>navigationParams</var>'s <span data-x="navigation-params-browsing-context">browsing
context</span>'s <span>session history</span>.</p></li>

domfarolino marked this conversation as resolved.
Show resolved Hide resolved
<li><p><span>Traverse the history</span> to the <span>current entry</span>.</p></li>
<li><p><span>Traverse the history</span> to <var>newEntry</var> with <var
data-x="traverse-history-hh">historyHandling</var> set to "<code
data-x="hh-replace">replace</code>".</p></li>
domfarolino marked this conversation as resolved.
Show resolved Hide resolved
</ol>
</dd>

<dt>Otherwise, if <var>navigationParams</var>'s <span data-x="navigation-params-hh">history
handling</span> is "<code data-x="hh-replace">replace</code>"</dt>

domfarolino marked this conversation as resolved.
Show resolved Hide resolved
<dd>
<ol>
<li><p>Let <var>newEntry</var> be a new <span>session history entry</span> whose <span
data-x="she-url">URL</span> is <var>newDocument</var>'s <span
domfarolino marked this conversation as resolved.
Show resolved Hide resolved
data-x="concept-document-url">URL</span> and <span data-x="she-document">document</span> is
<var>newDocument</var>.</p></li>

<li><p>Insert <var>newEntry</var> after the <span>current entry</span> in
<var>navigationParams</var>'s <span data-x="navigation-params-browsing-context">browsing
context</span>'s <span>session history</span>.</p></li>

<li><p><span>Traverse the history</span> to <var>newEntry</var> with
<var data-x="traverse-history-hh">historyHandling</var> set to "<code
data-x="hh-replace">replace</code>".</p></li>
</ol>
</dd>

<dt>Otherwise</dt>

<dd>
<ol>
<li><p>Assert: <var>navigationParams</var>'s <span data-x="navigation-params-hh">history
handling</span> is "<code data-x="hh-default">default</code>".</p></li>

<li>
<p>Remove all the entries in the <span>session history</span> after the <span>current
entry</span>. If the <span>current entry</span> is the last entry in the session history,
Expand All @@ -85198,9 +85233,7 @@ new PaymentRequest(&hellip;); // Allowed to use
data-x="concept-document-url">URL</span> and <span data-x="she-document">document</span> is
<var>newDocument</var>.</p></li>

<li><p><span>Traverse the history</span> to the new entry, with <var
data-x="traverse-history-hh">historyHandling</var> set to <var>navigationParams</var>'s <span
data-x="navigation-params-hh">history handling</span>.</p></li>
<li><p><span>Traverse the history</span> to the new entry.</p></li>
</ol>
</dd>
</dl>
Expand Down Expand Up @@ -85273,7 +85306,8 @@ new PaymentRequest(&hellip;); // Allowed to use

<p>After creating the <code>Document</code> object, but before any script execution, certainly
before the parser <span data-x="stop parsing">stops</span>, the user agent must <span>update the
session history with the new page</span> given <var>navigationParams</var>.</p>
session history with the new page</span> given <var>navigationParams</var> and the newly-created
<code>Document</code>.</p>



Expand All @@ -85298,9 +85332,10 @@ new PaymentRequest(&hellip;); // Allowed to use
character encoding.</p>

<p>Then, with the newly created <code>Document</code>, the user agent must <span>update the
session history with the new page</span> given <var>navigationParams</var>. User agents may do
this before the complete document has been parsed (thus achieving <i>incremental rendering</i>),
and must do this before any scripts are to be executed.</p>
session history with the new page</span> given <var>navigationParams</var> and the newly-created
<code>Document</code>. User agents may do this before the complete document has been parsed (thus
achieving <i>incremental rendering</i>), and must do this before any scripts are to be
executed.</p>

<p>Error messages from the parse process (e.g., XML namespace well-formedness errors) may be
reported inline by mutating the <code>Document</code>.</p>
Expand Down Expand Up @@ -85347,7 +85382,7 @@ new PaymentRequest(&hellip;); // Allowed to use

<p>After creating the <code>Document</code> object, but potentially before the page has finished
parsing, the user agent must <span>update the session history with the new page</span> given
<var>navigationParams</var>.</p>
<var>navigationParams</var> and the newly-created <code>Document</code>.</p>

<p>User agents may add content to the <code>head</code> element of the <code>Document</code>,
e.g., linking to a style sheet, providing script, or giving the document a <code>title</code>.</p>
Expand Down Expand Up @@ -85435,7 +85470,7 @@ new PaymentRequest(&hellip;); // Allowed to use

<p>After creating the <code>Document</code> object, but potentially before the page has finished
fully loading, the user agent must <span>update the session history with the new page</span> given
<var>navigationParams</var>.</p>
<var>navigationParams</var> and the newly-created <code>Document</code>.</p>

<p>User agents may add content to the <code>head</code> element of the <code>Document</code>, or
attributes to the element <var>host element</var>, e.g., to link to a style sheet, to provide a
Expand Down Expand Up @@ -85480,7 +85515,7 @@ new PaymentRequest(&hellip;); // Allowed to use

<p>After creating the <code>Document</code> object, but potentially before the page has finished
fully loading, the user agent must <span>update the session history with the new page</span> given
<var>navigationParams</var>.</p>
<var>navigationParams</var> and the newly-created <code>Document</code>.</p>

<p>User agents may add content to the <code>head</code> element of the <code>Document</code>, or
attributes to the <code>embed</code> element, e.g. to link to a style sheet or to give the
Expand Down Expand Up @@ -85536,7 +85571,7 @@ new PaymentRequest(&hellip;); // Allowed to use

<p>After creating the <code>Document</code> object, but potentially before the page has been
completely set up, the user agent must <span>update the session history with the new page</span>
given <var>navigationParams</var>.</p>
given <var>navigationParams</var> and the newly-created <code>Document</code>.</p>



Expand Down Expand Up @@ -85716,16 +85751,16 @@ new PaymentRequest(&hellip;); // Allowed to use
<p>If <var>entry</var>'s <span data-x="she-document">document</span> is null, then:</p>

<ol>
<li><p>Assert: <var>historyHandling</var> is "<code
data-x="hh-default">default</code>".</p></li>

<li><p>Let <var>request</var> be a new <span data-x="concept-request">request</span> whose
<span data-x="concept-request-url">url</span> is <var>entry</var>'s <span
data-x="she-url">URL</span>.</p></li>

<li><p>If <var>explicitHistoryNavigation</var> is true, then set <var>request</var>'s <span
data-x="concept-request-history-navigation-flag">history-navigation flag</span>.</p></li>

<li><p>Assert: <var>historyHandling</var> is not "<code
data-x="hh-replace">replace</code>".</p></li>

<li>
<p><span>Navigate</span><!--DONAV history traversal after eviction--> the <span>browsing
context</span> to <var>request</var> with <var data-x="navigation-hh">historyHandling</var>
Expand Down Expand Up @@ -85758,7 +85793,9 @@ new PaymentRequest(&hellip;); // Allowed to use
data-x="she-document">document</span>.</p></li>

<li><p>If <var>newDocument</var> is different than the <span>current entry</span>'s <span
data-x="she-document">document</span>, then:</p>
data-x="she-document">document</span>, or <var>historyHandling</var> is "<code
data-x="hh-entry-update">entry update</code>" or "<code data-x="hh-reload">reload</code>",
then:</p>

<ol>
<li><p>Remove any <span data-x="concept-task">tasks</span> queued by the <span>history traversal
Expand Down