diff options
author | khorimoto@chromium.org <khorimoto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-28 19:00:06 +0000 |
---|---|---|
committer | khorimoto@chromium.org <khorimoto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-28 19:00:06 +0000 |
commit | 3d5f3c13150e5cb5fa1fd779d26d9bca9b49acd3 (patch) | |
tree | 5efc46e7ba6f76ad09de5e36111eeb9b02de61e8 /dbus/values_util.cc | |
parent | ec6be596c3dc1518057b9aa1c4965eefe0267f03 (diff) | |
download | chromium_src-3d5f3c13150e5cb5fa1fd779d26d9bca9b49acd3.zip chromium_src-3d5f3c13150e5cb5fa1fd779d26d9bca9b49acd3.tar.gz chromium_src-3d5f3c13150e5cb5fa1fd779d26d9bca9b49acd3.tar.bz2 |
Checked the return values of function calls.
CID=103653,103654,103655
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/9863050
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129454 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'dbus/values_util.cc')
-rw-r--r-- | dbus/values_util.cc | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/dbus/values_util.cc b/dbus/values_util.cc index e341ac2..96cc37b3 100644 --- a/dbus/values_util.cc +++ b/dbus/values_util.cc @@ -207,25 +207,29 @@ void AppendBasicTypeValueData(MessageWriter* writer, const base::Value& value) { switch (value.GetType()) { case base::Value::TYPE_BOOLEAN: { bool bool_value = false; - value.GetAsBoolean(&bool_value); + bool success = value.GetAsBoolean(&bool_value); + DCHECK(success); writer->AppendBool(bool_value); break; } case base::Value::TYPE_INTEGER: { int int_value = 0; - value.GetAsInteger(&int_value); + bool success = value.GetAsInteger(&int_value); + DCHECK(success); writer->AppendInt32(int_value); break; } case base::Value::TYPE_DOUBLE: { double double_value = 0; - value.GetAsDouble(&double_value); + bool success = value.GetAsDouble(&double_value); + DCHECK(success); writer->AppendDouble(double_value); break; } case base::Value::TYPE_STRING: { std::string string_value; - value.GetAsString(&string_value); + bool success = value.GetAsString(&string_value); + DCHECK(success); writer->AppendString(string_value); break; } |