Skip to content
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.

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

S13nServiceWorker: Fix navigation redirect related tests #7550

Closed
wants to merge 1 commit into from

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Oct 3, 2017

Fix the failure of navigation-redirect.https.html and navigation-redirect.https.html by:

  1. Set request/credentials/redirect mode in NavigationURLLoaderNetworkService.
  2. Set content-type in the python script not to rely on the mime sniffing.

Bug: 764224
Change-Id: I0b061c26e3bd0a5d7de9afa8d813439d977212bb

Reviewed-on: https://chromium-review.googlesource.com/697086
WPT-Export-Revision: e0daf3f1dd0e67b4f6caa80a378dfdde0d23ea6f

Fix the failure of navigation-redirect.https.html and navigation-redirect.https.html by:
1) Set request/credentials/redirect mode in NavigationURLLoaderNetworkService.
2) Set content-type in the python script not to rely on the mime sniffing.

Bug: 764224
Change-Id: I0b061c26e3bd0a5d7de9afa8d813439d977212bb
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

Already reviewed downstream.

@w3c-bots
Copy link

w3c-bots commented Oct 3, 2017

Build PASSED

Started: 2017-10-03 06:55:34
Finished: 2017-10-03 07:05:36

Failing Jobs

  • safari:10.0
  • MicrosoftEdge:14.14393

View more information about this build on:

@Hexcles
Copy link
Member

Hexcles commented Jun 22, 2018

No exportable changes.

@Hexcles Hexcles closed this Jun 22, 2018
@Hexcles Hexcles deleted the chromium-export-cl-697086 branch June 22, 2018 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants