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

Unified Diff: sync/api/attachments/attachment_downloader.h

Issue 356953009: Revert of Consolidate attachment URL construction. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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: sync/api/attachments/attachment_downloader.h
diff --git a/sync/api/attachments/attachment_downloader.h b/sync/api/attachments/attachment_downloader.h
index b2b2037fc7c53af699a83072bd402711fdaedf0f..718284cc930cd4f2e711fddacea83e9704e63326 100644
--- a/sync/api/attachments/attachment_downloader.h
+++ b/sync/api/attachments/attachment_downloader.h
@@ -41,7 +41,8 @@
const DownloadCallback& callback) = 0;
// Create instance of AttachmentDownloaderImpl.
- // |sync_service_url| is the URL of the sync service.
+ // |url_prefix| is the URL prefix (including trailing slash) to be used when
+ // downloading attachments.
//
// |url_request_context_getter| provides a URLRequestContext.
//
@@ -51,7 +52,7 @@
//
// |token_service_provider| provides an OAuth2 token service.
static scoped_ptr<AttachmentDownloader> Create(
- const GURL& sync_service_url,
+ const std::string& url_prefix,
const scoped_refptr<net::URLRequestContextGetter>&
url_request_context_getter,
const std::string& account_id,
« no previous file with comments | « chrome/browser/sync/profile_sync_components_factory_impl.cc ('k') | sync/api/attachments/attachment_downloader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698