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

Side by Side Diff: components/sync/core_impl/attachments/attachment_downloader.cc

Issue 2388973002: [Sync] Removing duplicate includes, part 2. (Closed)
Patch Set: Update for Max's 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/sync/core/attachments/attachment_downloader.h" 5 #include "components/sync/core/attachments/attachment_downloader.h"
6 6
7 #include <string>
8
9 #include "components/sync/core/attachments/attachment_downloader_impl.h" 7 #include "components/sync/core/attachments/attachment_downloader_impl.h"
10 8
11 namespace syncer { 9 namespace syncer {
12 10
13 AttachmentDownloader::~AttachmentDownloader() {} 11 AttachmentDownloader::~AttachmentDownloader() {}
14 12
15 // Factory function for creating AttachmentDownloaderImpl. 13 // Factory function for creating AttachmentDownloaderImpl.
16 // It is introduced to avoid SYNC_EXPORT-ing AttachmentDownloaderImpl since it 14 // It is introduced to avoid SYNC_EXPORT-ing AttachmentDownloaderImpl since it
17 // inherits from OAuth2TokenService::Consumer which is not exported. 15 // inherits from OAuth2TokenService::Consumer which is not exported.
18 std::unique_ptr<AttachmentDownloader> AttachmentDownloader::Create( 16 std::unique_ptr<AttachmentDownloader> AttachmentDownloader::Create(
19 const GURL& sync_service_url, 17 const GURL& sync_service_url,
20 const scoped_refptr<net::URLRequestContextGetter>& 18 const scoped_refptr<net::URLRequestContextGetter>&
21 url_request_context_getter, 19 url_request_context_getter,
22 const std::string& account_id, 20 const std::string& account_id,
23 const OAuth2TokenService::ScopeSet scopes, 21 const OAuth2TokenService::ScopeSet scopes,
24 const scoped_refptr<OAuth2TokenServiceRequest::TokenServiceProvider>& 22 const scoped_refptr<OAuth2TokenServiceRequest::TokenServiceProvider>&
25 token_service_provider, 23 token_service_provider,
26 const std::string& store_birthday, 24 const std::string& store_birthday,
27 ModelType model_type) { 25 ModelType model_type) {
28 return std::unique_ptr<AttachmentDownloader>(new AttachmentDownloaderImpl( 26 return std::unique_ptr<AttachmentDownloader>(new AttachmentDownloaderImpl(
29 sync_service_url, url_request_context_getter, account_id, scopes, 27 sync_service_url, url_request_context_getter, account_id, scopes,
30 token_service_provider, store_birthday, model_type)); 28 token_service_provider, store_birthday, model_type));
31 } 29 }
32 30
33 } // namespace syncer 31 } // namespace syncer
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698