Merged

Conversation

Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>

Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>

I removed one extra line url.fragment().is_none() which resulted in these failures.
But I feel test is not aligned with spec:

	promise_test(t => {
	const url = URL.createObjectURL(blob);
	URL.revokeObjectURL(url + '#fragment');

	return fetch_should_succeed(t, url).then(text => {
	assert_equals(text, blob_contents);
	});
	}, 'Only exact matches should revoke URLs, using ' + fetch_method);

Per spec:

parsing then says:
...
Set url’s blob URL entry to the result of resolving the blob URL url, ...
which goes back to FileAPI.

resolving the blob URL step 3:
Let url string be the result of serializing url with the exclude fragment flag set. (back to whatwg again)

But with "exclude fragment flag set", we should not care about fragment?

Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>