| Index: extensions/browser/api/runtime/runtime_api.cc
|
| diff --git a/extensions/browser/api/runtime/runtime_api.cc b/extensions/browser/api/runtime/runtime_api.cc
|
| index 9dc3febdc000555f36c07aac866c04fad48fa944..94fb0121f1e7ebb38a2a117029f3e9aa9fd2ea79 100644
|
| --- a/extensions/browser/api/runtime/runtime_api.cc
|
| +++ b/extensions/browser/api/runtime/runtime_api.cc
|
| @@ -460,14 +460,12 @@ ExtensionFunction::ResponseAction RuntimeRequestUpdateCheckFunction::Run() {
|
| void RuntimeRequestUpdateCheckFunction::CheckComplete(
|
| const RuntimeAPIDelegate::UpdateCheckResult& result) {
|
| if (result.success) {
|
| - base::ListValue* results = new base::ListValue;
|
| - results->AppendString(result.response);
|
| base::DictionaryValue* details = new base::DictionaryValue;
|
| - results->Append(details);
|
| details->SetString("version", result.version);
|
| - Respond(MultipleArguments(results));
|
| + Respond(TwoArguments(new base::StringValue(result.response), details));
|
| } else {
|
| - Respond(SingleArgument(new base::StringValue(result.response)));
|
| + // HMM(kalman): Why does !success not imply Error()?
|
| + Respond(OneArgument(new base::StringValue(result.response)));
|
| }
|
| }
|
|
|
| @@ -489,8 +487,8 @@ ExtensionFunction::ResponseAction RuntimeGetPlatformInfoFunction::Run() {
|
| ->GetPlatformInfo(&info)) {
|
| return RespondNow(Error(kPlatformInfoUnavailable));
|
| }
|
| - return RespondNow(MultipleArguments(
|
| - runtime::GetPlatformInfo::Results::Create(info).release()));
|
| + return RespondNow(
|
| + ArgumentList(runtime::GetPlatformInfo::Results::Create(info)));
|
| }
|
|
|
| ExtensionFunction::ResponseAction
|
| @@ -511,7 +509,7 @@ RuntimeGetPackageDirectoryEntryFunction::Run() {
|
| base::DictionaryValue* dict = new base::DictionaryValue();
|
| dict->SetString("fileSystemId", filesystem_id);
|
| dict->SetString("baseName", relative_path);
|
| - return RespondNow(SingleArgument(dict));
|
| + return RespondNow(OneArgument(dict));
|
| }
|
|
|
| } // namespace extensions
|
|
|