diff options
| author | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-19 14:25:18 +0000 |
|---|---|---|
| committer | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-04-19 14:25:18 +0000 |
| commit | 135f59435e8aa4ee4ef60014569ab50fd5ff7775 (patch) | |
| tree | b56db7ff1af4b410d339ee849b756d5071279c40 /ppapi/thunk/ppb_gamepad_thunk.cc | |
| parent | a2553844dcff08889fb953ed629f46d0221a6ea1 (diff) | |
| download | chromium_src-135f59435e8aa4ee4ef60014569ab50fd5ff7775.zip chromium_src-135f59435e8aa4ee4ef60014569ab50fd5ff7775.tar.gz chromium_src-135f59435e8aa4ee4ef60014569ab50fd5ff7775.tar.bz2 | |
Pepper: Simplify idl_thunk implementation.
This gets rid of some code duplication, and rewrites functions in thunk files
which return void to an alternate, equivalent form.
BUG=
Review URL: https://codereview.chromium.org/14161017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@195180 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/thunk/ppb_gamepad_thunk.cc')
| -rw-r--r-- | ppapi/thunk/ppb_gamepad_thunk.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ppapi/thunk/ppb_gamepad_thunk.cc b/ppapi/thunk/ppb_gamepad_thunk.cc index e2f0160..e36af34 100644 --- a/ppapi/thunk/ppb_gamepad_thunk.cc +++ b/ppapi/thunk/ppb_gamepad_thunk.cc @@ -23,11 +23,11 @@ namespace { void Sample(PP_Instance instance, struct PP_GamepadsSampleData* data) { VLOG(4) << "PPB_Gamepad::Sample()"; EnterInstanceAPI<PPB_Gamepad_API> enter(instance); - if (enter.succeeded()) { - enter.functions()->Sample(instance, data); + if (enter.failed()) { + memset(data, 0, sizeof(*data)); return; } - memset(data, 0, sizeof(*data)); + enter.functions()->Sample(instance, data); } const PPB_Gamepad_1_0 g_ppb_gamepad_thunk_1_0 = { |