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

Unified Diff: sync/internal_api/public/attachments/attachment_downloader_impl.h

Issue 458753006: Fix use after free bug by calling GetTokenService in Core's ctor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update TSP comment (CANDIDATE). Created 6 years, 4 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/internal_api/public/attachments/attachment_downloader_impl.h
diff --git a/sync/internal_api/public/attachments/attachment_downloader_impl.h b/sync/internal_api/public/attachments/attachment_downloader_impl.h
index cf620d151081b3d6c8145c091312b586f8775070..8b62ec34dc1604255279e8175057d933bd58e775 100644
--- a/sync/internal_api/public/attachments/attachment_downloader_impl.h
+++ b/sync/internal_api/public/attachments/attachment_downloader_impl.h
@@ -36,7 +36,7 @@ class AttachmentDownloaderImpl : public AttachmentDownloader,
url_request_context_getter,
const std::string& account_id,
const OAuth2TokenService::ScopeSet& scopes,
- scoped_ptr<OAuth2TokenServiceRequest::TokenServiceProvider>
+ const scoped_refptr<OAuth2TokenServiceRequest::TokenServiceProvider>&
token_service_provider);
virtual ~AttachmentDownloaderImpl();
@@ -73,7 +73,7 @@ class AttachmentDownloaderImpl : public AttachmentDownloader,
std::string account_id_;
OAuth2TokenService::ScopeSet oauth2_scopes_;
- scoped_ptr<OAuth2TokenServiceRequest::TokenServiceProvider>
+ scoped_refptr<OAuth2TokenServiceRequest::TokenServiceProvider>
token_service_provider_;
scoped_ptr<OAuth2TokenService::Request> access_token_request_;

Powered by Google App Engine
This is Rietveld 408576698