Skip to content

[css-syntax-3][css-values-3][css-cascade-4][css-fonts-4] Formalize fetching and resource timing reporting #6715

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

Merged
merged 14 commits into from
Nov 3, 2021
Merged

[css-syntax-3][css-values-3][css-cascade-4][css-fonts-4] Formalize fetching and resource timing reporting #6715

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
64becb5
Formalize fetching and resource timing reporting
noamr Oct 5, 2021
6865584
Use more precise terms for same-origin
noamr Oct 5, 2021
e47f6fe
Update css-fonts-4/Overview.bs
noamr Oct 6, 2021
05e285e
Update css-cascade-4/Overview.bs
noamr Oct 6, 2021
99b8996
Update css-values-4/Overview.bs
noamr Oct 6, 2021
ab66419
Apply suggestions from code review
noamr Oct 6, 2021
34002f2
Nits
noamr Oct 6, 2021
ffc8bf7
Numbering
noamr Oct 11, 2021
69379fb
Tighten fetch integration, add shape support
noamr Oct 11, 2021
6dc96e0
Shapes, images, colors
noamr Oct 12, 2021
e053f32
Correct referrer
noamr Oct 12, 2021
cdbbf63
Switch colorprofile fetching to cors
noamr Oct 13, 2021
e610888
Remove referer thing for now
noamr Oct 28, 2021
45e648d
Revert "Remove referer thing for now"
noamr Oct 29, 2021
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
Prev Previous commit
Next Next commit
Numbering
  • Loading branch information
@noamr
noamr committed Oct 11, 2021
commit ffc8bf7550261efef26ebc395e0563ddcb484027
22 changes: 11 additions & 11 deletions css-values-4/Overview.bs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1009,29 +1009,29 @@ URL processing model</h4>

5. Let |referrer| be |documentBase|.

6. Let |handleResourceFetchDone| be to do nothing.
6. If |base| is null, set |base| to |documentBase|.

7. If |base| is null, set |base| to |documentBase|.

8. Let |parsedUrl| be the result of the [=URL parser=] steps with |url| and |base|.
7. Let |parsedUrl| be the result of the [=URL parser=] steps with |url| and |base|.
If the algorithm returns an error, return.

9. If |corsMode| is "cors", set |referrer| to |sheet|'s
8. If |corsMode| is "cors", set |referrer| to |sheet|'s
<a spec=cssom>location</a>. [[CSSOM]]

10. Let |req| be a new [=/request=] whose [=request/url=] is |parsedUrl|, whose
9. Let |req| be a new [=/request=] whose [=request/url=] is |parsedUrl|, whose
[=request/destination=] is |destination|, [=request/mode=] is |mode|, [=request/origin=]
is |environmentSettings|'s [=environment settings object/origin=],
[=request/credentials mode=] is "same-origin", [=request/use-url-credentials flag=] is
set, and whose [=request/header list=] is a [=/header list=] containing a [=header=]
with its [=header/name=] set to "referrer" and its [=header/value=] set to |referrer|.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is somewhat wrong. You want to set the referrer field on the request. That will set the Referer (note, one r less) header. But good that it sets it explicitly though as CSS cannot take it from the global.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, fixed.


11. If |sheet|'s <a spec=cssom>origin-clean flag</a> is set, set |handleResourceFetchDone|
given [=/response=] |res| to [=finalize and report timing=] with |res|, |global|, and
<code>"css"</code>. [[CSSOM]]
10. [=/Fetch=] |req|, with |taskDestination| set to |global|, and |processResponse| set to
the following steps given [=/response=] |res|:

1. If |sheet|'s <a spec=cssom>origin-clean flag</a> is set, set |handleResourceFetchDone|
given [=/response=] |res| to [=finalize and report timing=] with |res|, |global|, and
<code>"css"</code>. [[CSSOM]]

12. [=/Fetch=] |req|, with |processResponseDone| set to |handleResourceFetchDone|,
|taskDestination| set to |global|, and |processResponse| set to |processResponse|.
2. Call |processResponseDone| with |res|.
</div>

<h2 id="numeric-types">
Expand Down