Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(194)

Unified Diff: third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/fetch.js

Issue 2783623003: Fetch API: Don't exclude the URL fragment in Request#url (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/fetch.js
diff --git a/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/fetch.js b/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/fetch.js
index e6f00e2e8e7e500ff01f2484d578d0a526036f0a..b84306cc42f742724ca6be5807f431b246c45290 100644
--- a/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/fetch.js
+++ b/third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/fetch.js
@@ -167,10 +167,10 @@ promise_test(function(t) {
var request = new Request(
'/fetch/resources/fetch-status.php?status=200#fragment');
- // The url attribute's getter must return request's url,
- // serialized with the exclude fragment flag set.
+ // The url attribute's getter must return request's url with the fragment
+ // flag set.
falken 2017/03/31 06:40:49 There's no "fragment flag", it's just the "exclude
yiyix 2017/03/31 07:24:02 True, the term "flag set" is not used in the spec.
assert_equals(request.url,
- BASE_ORIGIN + '/fetch/resources/fetch-status.php?status=200');
+ BASE_ORIGIN + '/fetch/resources/fetch-status.php?status=200#fragment');
return fetch(request)
.then(function(response) {

Powered by Google App Engine
This is Rietveld 408576698