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

Unified Diff: google_apis/drive/test_util.cc

Issue 2481923002: [WIP] make GURL::path() return a StringPiece (Closed)
Patch Set: thanks asan Created 4 years, 1 month 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
« no previous file with comments | « google_apis/drive/drive_api_requests_unittest.cc ('k') | google_apis/gaia/fake_gaia.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: google_apis/drive/test_util.cc
diff --git a/google_apis/drive/test_util.cc b/google_apis/drive/test_util.cc
index a6497ca981ffd919a77e6b5e39287bcf7c89e743..ffead523d10de0574795842892183a7019c34d46 100644
--- a/google_apis/drive/test_util.cc
+++ b/google_apis/drive/test_util.cc
@@ -116,8 +116,10 @@ std::unique_ptr<net::test_server::HttpResponse> HandleDownloadFileRequest(
GURL absolute_url = base_url.Resolve(request.relative_url);
std::string remaining_path;
- if (!RemovePrefix(absolute_url.path(), "/files/", &remaining_path))
+ if (!RemovePrefix(absolute_url.path().as_string(), "/files/",
+ &remaining_path)) {
return std::unique_ptr<net::test_server::HttpResponse>();
+ }
return CreateHttpResponseFromFile(GetTestFilePath(remaining_path));
}
« no previous file with comments | « google_apis/drive/drive_api_requests_unittest.cc ('k') | google_apis/gaia/fake_gaia.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698