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

Issue 9981003: Merge 112485 - AssociatedURLLoader does not support Cross Origin Redirects when using (Closed)

Created:
8 years, 8 months ago by bbudge
Modified:
8 years, 8 months ago
Reviewers:
bbudge
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1084/
Visibility:
Public.

Description

Merge 112485 - AssociatedURLLoader does not support Cross Origin Redirects when using Access Control. https://bugs.webkit.org/show_bug.cgi?id=82354 AssociatedURLLoader's internal adapter now overrides didFailRedirectCheck, which cancels the load, causing didFail to notify the client that the load failed. AssociatedURLLoaderTest adds test cases for CORS requests that receive redirects and pass or fail the redirect access check. Reviewed by Adam Barth. * src/AssociatedURLLoader.cpp: (AssociatedURLLoader::ClientAdapter): (WebKit::AssociatedURLLoader::ClientAdapter::didFailRedirectCheck): (WebKit): TBR=bbudge@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=113197

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M Source/WebKit/chromium/src/AssociatedURLLoader.cpp View 2 chunks +6 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
bbudge
8 years, 8 months ago (2012-04-04 16:01:24 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698