Skip to content

Commit 8b57aea

Browse files
committed
Address review feedback
1 parent fd3d0f8 commit 8b57aea

File tree

1 file changed

+16
-11
lines changed

1 file changed

+16
-11
lines changed

index.bs

Lines changed: 16 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1512,20 +1512,25 @@ A [=blob URL store=] is a [=map=]
15121512
where [=map/keys=] are [=valid URL strings=]
15131513
and [=map/values=] are [=blob URL Entries=].
15141514

1515+
A <dfn export>blob URL entry</dfn> consists of an <i>object</i> (of type {{Blob}} or
1516+
{{MediaSource}}), and an <dfn export for="blob URL entry">environment</dfn> (an
1517+
[=environment settings object=]).
1518+
1519+
Note: Specifications must use the [=obtain a blob object=] algorithm to access a
1520+
[=blob URL entry=]'s <i>object</i>.
1521+
15151522
[=map/Keys=] in the [=blob URL store=] (also known as <dfn lt="blob URL|object URL" export>blob URLs</dfn>)
15161523
are [=valid URL strings=] that when [=URL parser|parsed=]
15171524
result in a [=/URL=] with a [=url/scheme=] equal to "`blob`",
15181525
an [=empty host=], and a [=url/path=] consisting of one element itself also a [=valid URL string=].
15191526

1520-
A <dfn export>blob URL entry</dfn> is used to store an object of type {{Blob}} or {{MediaSource}}.
1521-
1522-
A [=blob URL entry=] has an <dfn export for="blob URL entry">environment</dfn> (an [=environment settings object=]).
1523-
15241527
<div algorithm="obtainBlobObject">
1525-
To <dfn export id=blob-url-obtain-object>obtain a blob object</dfn> given a [=blob URL entry=] |blobUrlEntry|, an [=environment=] |environment|, and an optional boolean |isNavigation| (default false):
1528+
To <dfn export id=blob-url-obtain-object>obtain a blob object</dfn> given a [=blob URL entry=]
1529+
|blobUrlEntry|, an [=environment=] or the string "`navigation`" |environment|:
15261530

15271531
1. Let |isAuthorized| be true.
1528-
1. If |isNavigation| is false, let |isAuthorized| be the result of [=checking for same-partition blob URL usage=] with |blobUrlEntry| and |environment|.
1532+
1. If |environment| is not the string "`navigation`", then set |isAuthorized| to the result of
1533+
[=checking for same-partition blob URL usage=] with |blobUrlEntry| and |environment|.
15291534
1. If |isAuthorized| is false, then return failure.
15301535
1. Return |blobUrlEntry|'s object.
15311536

@@ -1656,12 +1661,12 @@ return the result of [=adding an entry to the blob URL store=] for |obj|.
16561661
<div algorithm="revokeObjectURL">
16571662
The <dfn method for=URL id="dfn-revokeObjectURL">revokeObjectURL(|url|)</dfn> static method must run these steps:
16581663

1659-
1. Let |url record| be the result of [=URL parser|parsing=] |url|.
1660-
1. If |url record|'s [=url/scheme=] is not "`blob`", return.
1661-
1. Let |entry| be |url record|'s [=blob URL entry=].
1662-
1. If |entry| is null, return.
1664+
1. Let |urlRecord| be the result of [=URL parser|parsing=] |url|.
1665+
1. If |urlRecord|'s [=url/scheme=] is not "`blob`", return.
1666+
1. Let |entry| be |urlRecord|'s [=blob URL entry=].
1667+
1. If |entry| is null, then return.
16631668
1. Let |isAuthorized| be the result of [=checking for same-partition blob URL usage=] with |entry| and the [=current settings object=].
1664-
1. If |isAuthorized| is false, return.
1669+
1. If |isAuthorized| is false, then return.
16651670
1. [=Remove an entry from the Blob URL Store=] for |url|.
16661671

16671672
Note: This means that rather than throwing some kind of error, attempting to revoke a URL that isn't registered or that was registered from an environment in a different storage partition will silently fail.

0 commit comments

Comments
 (0)