Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(303)

Unified Diff: ppapi/thunk/ppb_widget_dev_thunk.cc

Issue 14161017: Pepper: Simplify idl_thunk implementation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: on_failure comment nit Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ppapi/thunk/ppb_url_loader_trusted_thunk.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/thunk/ppb_widget_dev_thunk.cc
diff --git a/ppapi/thunk/ppb_widget_dev_thunk.cc b/ppapi/thunk/ppb_widget_dev_thunk.cc
index 3a6fab943770eb616874a0fbb9742c17d7ddf943..2232371a69ee664b7511c85b144296c22c328416 100644
--- a/ppapi/thunk/ppb_widget_dev_thunk.cc
+++ b/ppapi/thunk/ppb_widget_dev_thunk.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-// From dev/ppb_widget_dev.idl modified Thu Dec 20 13:10:26 2012.
+// From dev/ppb_widget_dev.idl modified Tue Apr 16 11:25:44 2013.
#include "ppapi/c/dev/ppb_widget_dev.h"
#include "ppapi/c/pp_errors.h"
@@ -53,15 +53,17 @@ PP_Bool GetLocation(PP_Resource widget, struct PP_Rect* location) {
void SetLocation(PP_Resource widget, const struct PP_Rect* location) {
VLOG(4) << "PPB_Widget_Dev::SetLocation()";
EnterResource<PPB_Widget_API> enter(widget, false);
- if (enter.succeeded())
- enter.object()->SetLocation(location);
+ if (enter.failed())
+ return;
+ enter.object()->SetLocation(location);
}
void SetScale(PP_Resource widget, float scale) {
VLOG(4) << "PPB_Widget_Dev::SetScale()";
EnterResource<PPB_Widget_API> enter(widget, false);
- if (enter.succeeded())
- enter.object()->SetScale(scale);
+ if (enter.failed())
+ return;
+ enter.object()->SetScale(scale);
}
const PPB_Widget_Dev_0_3 g_ppb_widget_dev_thunk_0_3 = {
« no previous file with comments | « ppapi/thunk/ppb_url_loader_trusted_thunk.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698