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

Unified Diff: components/sync/engine/attachments/attachment_uploader.cc

Issue 2399953002: [Sync] Move attachments code out of core/. (Closed)
Patch Set: Address comments. Created 4 years, 2 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: components/sync/engine/attachments/attachment_uploader.cc
diff --git a/components/sync/core_impl/attachments/attachment_downloader.cc b/components/sync/engine/attachments/attachment_uploader.cc
similarity index 52%
rename from components/sync/core_impl/attachments/attachment_downloader.cc
rename to components/sync/engine/attachments/attachment_uploader.cc
index 9483fbf26c14756e17dd55a90bb444bd50609321..fedc9aec70e56d10581add4756c69efb3310d542 100644
--- a/components/sync/core_impl/attachments/attachment_downloader.cc
+++ b/components/sync/engine/attachments/attachment_uploader.cc
@@ -2,18 +2,16 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "components/sync/core/attachments/attachment_downloader.h"
+#include "components/sync/engine/attachments/attachment_uploader.h"
-#include "components/sync/core/attachments/attachment_downloader_impl.h"
+#include "base/memory/ptr_util.h"
+#include "components/sync/engine_impl/attachments/attachment_uploader_impl.h"
namespace syncer {
-AttachmentDownloader::~AttachmentDownloader() {}
+AttachmentUploader::~AttachmentUploader() {}
-// Factory function for creating AttachmentDownloaderImpl.
-// It is introduced to avoid SYNC_EXPORT-ing AttachmentDownloaderImpl since it
-// inherits from OAuth2TokenService::Consumer which is not exported.
-std::unique_ptr<AttachmentDownloader> AttachmentDownloader::Create(
+std::unique_ptr<AttachmentUploader> AttachmentUploader::Create(
const GURL& sync_service_url,
const scoped_refptr<net::URLRequestContextGetter>&
url_request_context_getter,
@@ -23,9 +21,9 @@ std::unique_ptr<AttachmentDownloader> AttachmentDownloader::Create(
token_service_provider,
const std::string& store_birthday,
ModelType model_type) {
- return std::unique_ptr<AttachmentDownloader>(new AttachmentDownloaderImpl(
+ return base::MakeUnique<AttachmentUploaderImpl>(
sync_service_url, url_request_context_getter, account_id, scopes,
- token_service_provider, store_birthday, model_type));
+ token_service_provider, store_birthday, model_type);
}
} // namespace syncer
« no previous file with comments | « components/sync/engine/attachments/attachment_uploader.h ('k') | components/sync/engine/attachments/attachment_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698