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

Unified Diff: chrome/browser/chromeos/app_mode/kiosk_app_data.cc

Issue 59663002: Switch the WebstoreDataFetcherDelegate::OnWebstoreResponseParseSuccess interface to scoped_ptr. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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
Index: chrome/browser/chromeos/app_mode/kiosk_app_data.cc
diff --git a/chrome/browser/chromeos/app_mode/kiosk_app_data.cc b/chrome/browser/chromeos/app_mode/kiosk_app_data.cc
index 63f10be6163de3d61f4d4d845662cd1a56d1a4c4..0aaa29e46274e44705e0b18354b0fb1de24870c4 100644
--- a/chrome/browser/chromeos/app_mode/kiosk_app_data.cc
+++ b/chrome/browser/chromeos/app_mode/kiosk_app_data.cc
@@ -409,28 +409,27 @@ void KioskAppData::OnWebstoreRequestFailure() {
}
void KioskAppData::OnWebstoreResponseParseSuccess(
- base::DictionaryValue* webstore_data) {
+ scoped_ptr<base::DictionaryValue> webstore_data) {
// Takes ownership of |webstore_data|.
- scoped_ptr<base::DictionaryValue> data(webstore_data);
-
webstore_fetcher_.reset();
std::string manifest;
- if (!CheckResponseKeyValue(data.get(), kManifestKey, &manifest))
+ if (!CheckResponseKeyValue(webstore_data.get(), kManifestKey, &manifest))
return;
- if (!CheckResponseKeyValue(data.get(), kLocalizedNameKey, &name_))
+ if (!CheckResponseKeyValue(webstore_data.get(), kLocalizedNameKey, &name_))
return;
std::string icon_url_string;
- if (!CheckResponseKeyValue(data.get(), kIconUrlKey, &icon_url_string))
+ if (!CheckResponseKeyValue(webstore_data.get(), kIconUrlKey,
+ &icon_url_string))
return;
GURL icon_url = GURL(extension_urls::GetWebstoreLaunchURL()).Resolve(
icon_url_string);
if (!icon_url.is_valid()) {
LOG(ERROR) << "Webstore response error (icon url): "
- << ValueToString(data.get());
+ << ValueToString(webstore_data.get());
OnWebstoreResponseParseFailure(kInvalidWebstoreResponseError);
return;
}
« no previous file with comments | « chrome/browser/chromeos/app_mode/kiosk_app_data.h ('k') | chrome/browser/extensions/webstore_data_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698