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

Unified Diff: chrome/browser/extensions/api/cookies/cookies_api.cc

Issue 10694106: Added support for multiple parameters to Extension API callbacks. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Synced. Created 8 years, 5 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: chrome/browser/extensions/api/cookies/cookies_api.cc
diff --git a/chrome/browser/extensions/api/cookies/cookies_api.cc b/chrome/browser/extensions/api/cookies/cookies_api.cc
index cb17f3096c346c74a6b0b309e59d40a4fd076692..58f928e91fb9ba7571e79deb851d498a13878b9f 100644
--- a/chrome/browser/extensions/api/cookies/cookies_api.cc
+++ b/chrome/browser/extensions/api/cookies/cookies_api.cc
@@ -230,15 +230,14 @@ void GetCookieFunction::GetCookieCallback(const net::CookieList& cookie_list) {
// CookieMonster returns them in canonical order (longest path, then
// earliest creation time).
if (it->Name() == name_) {
- result_.reset(
- cookies_helpers::CreateCookieValue(*it, store_id_));
+ SetResult(cookies_helpers::CreateCookieValue(*it, store_id_));
break;
}
}
// The cookie doesn't exist; return null.
if (it == cookie_list.end())
- result_.reset(Value::CreateNullValue());
+ SetResult(Value::CreateNullValue());
bool rv = BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE,
@@ -296,7 +295,7 @@ void GetAllCookiesFunction::GetAllCookiesCallback(
cookies_helpers::AppendMatchingCookiesToList(
cookie_list, store_id_, url_, details_,
GetExtension(), matching_list);
- result_.reset(matching_list);
+ SetResult(matching_list);
}
bool rv = BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE,
@@ -408,8 +407,7 @@ void SetCookieFunction::PullCookieCallback(const net::CookieList& cookie_list) {
// CookieMonster returns them in canonical order (longest path, then
// earliest creation time).
if (it->Name() == name_) {
- result_.reset(
- cookies_helpers::CreateCookieValue(*it, store_id_));
+ SetResult(cookies_helpers::CreateCookieValue(*it, store_id_));
break;
}
}
@@ -481,7 +479,7 @@ void RemoveCookieFunction::RemoveCookieCallback() {
resultDictionary->SetString(keys::kNameKey, name_);
resultDictionary->SetString(keys::kUrlKey, url_.spec());
resultDictionary->SetString(keys::kStoreIdKey, store_id_);
- result_.reset(resultDictionary);
+ SetResult(resultDictionary);
// Return to UI thread
bool rv = BrowserThread::PostTask(
@@ -536,7 +534,7 @@ bool GetAllCookieStoresFunction::RunImpl() {
cookies_helpers::CreateCookieStoreValue(
incognito_profile, incognito_tab_ids.release()));
}
- result_.reset(cookie_store_list);
+ SetResult(cookie_store_list);
return true;
}

Powered by Google App Engine
This is Rietveld 408576698