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

Unified Diff: chrome/browser/rlz/rlz_extension_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/rlz/rlz_extension_api.cc
diff --git a/chrome/browser/rlz/rlz_extension_api.cc b/chrome/browser/rlz/rlz_extension_api.cc
index e164506f7f3b7b64374541c6237d2ee797a17bb9..e61c8fd7fb205b060c760c99ed4d54925e994c00 100644
--- a/chrome/browser/rlz/rlz_extension_api.cc
+++ b/chrome/browser/rlz/rlz_extension_api.cc
@@ -113,7 +113,7 @@ bool RlzGetAccessPointRlzFunction::RunImpl() {
char rlz[rlz_lib::kMaxRlzLength + 1];
rlz_lib::GetAccessPointRlz(access_point, rlz, rlz_lib::kMaxRlzLength);
- result_.reset(Value::CreateStringValue(rlz));
+ SetResult(Value::CreateStringValue(rlz));
return true;
}
@@ -179,7 +179,7 @@ void RlzSendFinancialPingFunction::WorkOnWorkerThread() {
id_.c_str(), lang_.c_str(),
exclude_machine_id_);
- result_.reset(Value::CreateBooleanValue(sent));
+ SetResult(Value::CreateBooleanValue(sent));
bool post_task_result = content::BrowserThread::PostTask(
content::BrowserThread::UI, FROM_HERE,

Powered by Google App Engine
This is Rietveld 408576698