diff options
| author | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-21 09:30:02 +0000 |
|---|---|---|
| committer | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-21 09:30:02 +0000 |
| commit | 8a80c1742b4fd46fe81da90fc3423ed5e653045d (patch) | |
| tree | 79036931211109bd50797a352c16e1ab38e9fa8a /ppapi | |
| parent | 86ac34465941c72366ffdf7bcb5de8bdbfe6804f (diff) | |
| download | chromium_src-8a80c1742b4fd46fe81da90fc3423ed5e653045d.zip chromium_src-8a80c1742b4fd46fe81da90fc3423ed5e653045d.tar.gz chromium_src-8a80c1742b4fd46fe81da90fc3423ed5e653045d.tar.bz2 | |
Coverity: Add a missing return statement.
CID=101004,101997
BUG=none
TEST=none
Adding sehr, whom I suggest is much more familiar with this code.
Review URL: http://codereview.chromium.org/9005025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115290 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi')
| -rw-r--r-- | ppapi/native_client/src/trusted/plugin/scriptable_handle.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ppapi/native_client/src/trusted/plugin/scriptable_handle.cc b/ppapi/native_client/src/trusted/plugin/scriptable_handle.cc index ef88c72..c9549a5 100644 --- a/ppapi/native_client/src/trusted/plugin/scriptable_handle.cc +++ b/ppapi/native_client/src/trusted/plugin/scriptable_handle.cc @@ -356,7 +356,7 @@ void ScriptableHandle::SetProperty(const pp::Var& name, args, exception); std::string exception_string("NULL"); - if (exception != NULL) { + if (!exception->is_undefined()) { exception_string = exception->DebugString(); } PLUGIN_PRINTF(("ScriptableHandle::SetProperty (exception=%s)\n", @@ -406,7 +406,7 @@ pp::Var ScriptableHandle::Call(const pp::Var& name, PLUGIN_PRINTF(("ScriptableHandle::Call (name=%s, %"NACL_PRIuS " args)\n", name.DebugString().c_str(), args.size())); if (plugin_ == NULL) { - pp::Var(); + return pp::Var(); } if (name.is_undefined()) // invoke default return pp::Var(); |