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

Unified Diff: chrome/browser/extensions/api/identity/gaia_web_auth_flow.cc

Issue 23526048: Support URL fragment resolution againt non-hierarchical schemes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixes Created 7 years, 3 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: chrome/browser/extensions/api/identity/gaia_web_auth_flow.cc
diff --git a/chrome/browser/extensions/api/identity/gaia_web_auth_flow.cc b/chrome/browser/extensions/api/identity/gaia_web_auth_flow.cc
index 2ee70f3b9602efcf99a9dda8b3c0a8683bcf30bc..ff7302b685770d71609355167fd41aef9f506817 100644
--- a/chrome/browser/extensions/api/identity/gaia_web_auth_flow.cc
+++ b/chrome/browser/extensions/api/identity/gaia_web_auth_flow.cc
@@ -113,11 +113,11 @@ void GaiaWebAuthFlow::OnAuthFlowURLChange(const GURL& url) {
// interpreted as a path, including the fragment.
if (url.scheme() == redirect_scheme_ && !url.has_host() && !url.has_port() &&
- StartsWithASCII(url.path(), redirect_path_prefix_, true)) {
+ StartsWithASCII(url.Content(), redirect_path_prefix_, true)) {
web_flow_.release()->DetachDelegateAndDelete();
std::string fragment =
- url.path().substr(redirect_path_prefix_.length(), std::string::npos);
+ url.Content().substr(redirect_path_prefix_.length(), std::string::npos);
std::vector<std::pair<std::string, std::string> > pairs;
base::SplitStringIntoKeyValuePairs(fragment, '=', '&', &pairs);
std::string access_token;
@@ -172,4 +172,4 @@ scoped_ptr<WebAuthFlow> GaiaWebAuthFlow::CreateWebAuthFlow(GURL url) {
WebAuthFlow::INTERACTIVE));
}
-} // extensions
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698