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

Unified Diff: ppapi/shared_impl/ppp_instance_combined.cc

Issue 9348092: Revert 121901 - PPAPI: Add unlocking for PPP calls and callbacks. Add more locking. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 10 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/proxy/serialized_var_unittest.cc ('k') | ppapi/shared_impl/proxy_lock.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/shared_impl/ppp_instance_combined.cc
===================================================================
--- ppapi/shared_impl/ppp_instance_combined.cc (revision 121902)
+++ ppapi/shared_impl/ppp_instance_combined.cc (working copy)
@@ -1,9 +1,8 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
#include "ppapi/shared_impl/ppp_instance_combined.h"
-#include "ppapi/shared_impl/proxy_lock.h"
namespace ppapi {
@@ -27,36 +26,31 @@
uint32_t argc,
const char* argn[],
const char* argv[]) {
- return CallWhileUnlocked(instance_1_1_.DidCreate, instance, argc, argn, argv);
+ return instance_1_1_.DidCreate(instance, argc, argn, argv);
}
void PPP_Instance_Combined::DidDestroy(PP_Instance instance) {
- return CallWhileUnlocked(instance_1_1_.DidDestroy, instance);
+ return instance_1_1_.DidDestroy(instance);
}
void PPP_Instance_Combined::DidChangeView(PP_Instance instance,
PP_Resource view_changed_resource,
const struct PP_Rect* position,
const struct PP_Rect* clip) {
- if (instance_1_1_.DidChangeView) {
- CallWhileUnlocked(instance_1_1_.DidChangeView,
- instance,
- view_changed_resource);
- } else {
- CallWhileUnlocked(did_change_view_1_0_, instance, position, clip);
- }
+ if (instance_1_1_.DidChangeView)
+ instance_1_1_.DidChangeView(instance, view_changed_resource);
+ else
+ did_change_view_1_0_(instance, position, clip);
}
void PPP_Instance_Combined::DidChangeFocus(PP_Instance instance,
PP_Bool has_focus) {
- CallWhileUnlocked(instance_1_1_.DidChangeFocus, instance, has_focus);
+ instance_1_1_.DidChangeFocus(instance, has_focus);
}
PP_Bool PPP_Instance_Combined::HandleDocumentLoad(PP_Instance instance,
PP_Resource url_loader) {
- return CallWhileUnlocked(instance_1_1_.HandleDocumentLoad,
- instance,
- url_loader);
+ return instance_1_1_.HandleDocumentLoad(instance, url_loader);
}
} // namespace ppapi
« no previous file with comments | « ppapi/proxy/serialized_var_unittest.cc ('k') | ppapi/shared_impl/proxy_lock.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698