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

Unified Diff: services/authenticating_url_loader/authenticating_url_loader_impl.h

Issue 1146923003: Handle redirects in AuthenticatingURLLoaderImpl. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Response to review Created 5 years, 7 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: services/authenticating_url_loader/authenticating_url_loader_impl.h
diff --git a/services/authenticating_url_loader/authenticating_url_loader_impl.h b/services/authenticating_url_loader/authenticating_url_loader_impl.h
index e1e9d030b1355addaa66e3a8812e1458ed80cc12..a3ef28f4a90ab4d0ebb148bbeb798bf05b82ca0b 100644
--- a/services/authenticating_url_loader/authenticating_url_loader_impl.h
+++ b/services/authenticating_url_loader/authenticating_url_loader_impl.h
@@ -39,6 +39,7 @@ class AuthenticatingURLLoaderImpl : public AuthenticatingURLLoader,
// AuthenticatingURLLoader methods:
void Start(URLRequestPtr request,
const Callback<void(URLResponsePtr)>& callback) override;
+ void FollowRedirect(const Callback<void(URLResponsePtr)>& callback) override;
// ErrorHandler methods:
void OnConnectionError() override;
@@ -47,6 +48,8 @@ class AuthenticatingURLLoaderImpl : public AuthenticatingURLLoader,
void OnLoadComplete(URLResponsePtr response);
+ void FollowRedirectInternal();
+
void OnAccountSelected(String username, String error);
void OnOAuth2TokenReceived(String token, String error);
@@ -64,7 +67,7 @@ class AuthenticatingURLLoaderImpl : public AuthenticatingURLLoader,
Array<String> headers_;
String username_;
String token_;
- Callback<void(URLResponsePtr)> pending_start_callback_;
+ Callback<void(URLResponsePtr)> pending_request_callback_;
};
} // namespace mojo

Powered by Google App Engine
This is Rietveld 408576698