Permalink
Browse files

Editorial: Clarifying secure context integration with Shared Workers

This caused some confusion in a bug report. I think we could have avoided that
if we'd used a name other than isSecureContext for the variable, as it's
easily confused with an operation.
  • Loading branch information...
mikewest authored and domenic committed Nov 20, 2017
1 parent f8f9c6c commit 6f9d81839133072355c15efa4b934beaf85ea259
Showing with 6 additions and 4 deletions.
  1. +6 −4 source
View
10 source
@@ -97698,8 +97698,8 @@ interface <dfn>SharedWorker</dfn> : <span>EventTarget</span> {
<li><p>Assign <var>outside port</var> to the <code data-x="dom-SharedWorker-port">port</code>
attribute of <var>worker</var>.</p></li>
<li><p>Let <var>isSecureContext</var> be the result of executing <span>Is environment settings
object a secure context?</span> on <var>outside settings</var>.</p></li>
<li><p>Let <var>callerIsSecureContext</var> be the result of executing <span>Is environment
settings object a secure context?</span> on <var>outside settings</var>.</p></li>
<li>
<p><span>Enqueue the following steps</span> to the <span>shared worker manager</span>:</p>
@@ -97748,8 +97748,10 @@ interface <dfn>SharedWorker</dfn> : <span>EventTarget</span> {
<li><p>Let <var>settings object</var> be the <span>relevant settings object</span> for
<var>worker global scope</var>.</p></li>
<li><p>If the result of executing <span>Is environment settings object a secure
context?</span> on <var>settings object</var> is not <var>isSecureContext</var>, then
<li><p>Let <var>workerIsSecureContext</var> be the result of executing <span>Is environment
settings object a secure context?</span> on <var>settings object</var>.</p></li>
<li><p>If <var>workerIsSecureContext</var> is not <var>callerIsSecureContext</var>, then
<span>queue a task</span> to <span data-x="concept-event-fire">fire an event</span> named
<code data-x="event-error">error</code> at <var>worker</var> and abort these subsubsteps.
<ref spec="SECURE-CONTEXTS"></p></li>

0 comments on commit 6f9d818

Please sign in to comment.