Fix clang compile failure in chromeos::SMSClient

BUG=None
TEST=try -b linux_chromeos_clang

Review URL: https://chromiumcodereview.appspot.com/10535127

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141637 0039d316-1c4b-4281-b951-d872f2087c98
parent 11208904
......@@ -52,7 +52,8 @@ class SMSClientImpl : public SMSClient {
void OnGetAll(const GetAllCallback& callback, dbus::Response* response) {
if (!response) {
// Must invoke the callback, even if there is no message.
callback.Run(base::DictionaryValue());
base::DictionaryValue empty_dictionary;
callback.Run(empty_dictionary);
return;
}
dbus::MessageReader reader(response);
......@@ -60,7 +61,8 @@ class SMSClientImpl : public SMSClient {
base::DictionaryValue* dictionary_value = NULL;
if (!value.get() || !value->GetAsDictionary(&dictionary_value)) {
LOG(WARNING) << "Invalid response: " << response->ToString();
callback.Run(base::DictionaryValue());
base::DictionaryValue empty_dictionary;
callback.Run(empty_dictionary);
return;
}
callback.Run(*dictionary_value);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment