diff options
author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-06 22:55:47 +0000 |
---|---|---|
committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-06 22:55:47 +0000 |
commit | 6239d34076222cbfe1d42770c604822b0ba894f4 (patch) | |
tree | d873cdf138b002bb2d2d4a554ba4fcd210251760 /ppapi/thunk/enter.h | |
parent | c50a948dd9c07a71524949a28a5347e11d80da47 (diff) | |
download | chromium_src-6239d34076222cbfe1d42770c604822b0ba894f4.zip chromium_src-6239d34076222cbfe1d42770c604822b0ba894f4.tar.gz chromium_src-6239d34076222cbfe1d42770c604822b0ba894f4.tar.bz2 |
This implements the new system for Graphics2D only.
This works by adding a new thunk layer that will forward to an "API" that's either per-instance (function APIs) or per-resource (resource APIs). The proxying and such is then implemented in terms of this C++ API.
Ideally the trackers of the PP_Resource/PP_Instance -> object mapping would be shared between the plugin and renderer processes. To keep this patch under control, I did this as a virtual base class which is implemented by ppapi::proxy::PluginResourceTracker and webkit::ppapi::ResourceTracker. Later, the functionality of these objects should be shared in a common tracker class.
Still to do it a lot of cleanup and merging of things. Also, the namespaces are a bit out of control.
Review URL: http://codereview.chromium.org/6905088
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84519 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/thunk/enter.h')
-rw-r--r-- | ppapi/thunk/enter.h | 91 |
1 files changed, 91 insertions, 0 deletions
diff --git a/ppapi/thunk/enter.h b/ppapi/thunk/enter.h new file mode 100644 index 0000000..1a18cf6 --- /dev/null +++ b/ppapi/thunk/enter.h @@ -0,0 +1,91 @@ +// 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. + +#ifndef PPAPI_THUNK_ENTER_H_ +#define PPAPI_THUNK_ENTER_H_ + +#include "base/basictypes.h" +#include "ppapi/c/pp_resource.h" +#include "ppapi/proxy/interface_id.h" +#include "ppapi/shared_impl/function_group_base.h" +#include "ppapi/shared_impl/resource_object_base.h" +#include "ppapi/shared_impl/tracker_base.h" + +namespace ppapi { +namespace thunk { + +// EnterHost* helper objects: These objects wrap a call from the C PPAPI into +// the internal implementation. They make sure the lock is acquired and will +// automatically set up some stuff for you. +// +// You should always check whether the enter succeeded before using the object. +// If this fails, then the instance or resource ID supplied was invalid. +// +// The |report_error| arguments to the constructor should indicate if errors +// should be logged to the console. If the calling function expects that the +// input values are correct (the normal case), this should be set to true. In +// some case like |IsFoo(PP_Resource)| the caller is quersioning whether their +// handle is this type, and we don't want to report an error if it's not. +// +// Standalone functions: EnterHostFunction +// Automatically gets the implementation for the function API for the +// supplied PP_Instance. +// +// Resource member functions: EnterHostResource +// Automatically interprets the given PP_Resource as a resource ID and sets +// up the resource object for you. + +template<typename FunctionsT> +class EnterFunction { + public: + EnterFunction(PP_Instance instance, bool report_error) + : functions_(NULL) { + shared_impl::FunctionGroupBase* base = + shared_impl::TrackerBase::Get()->GetFunctionAPI( + instance, FunctionsT::interface_id); + if (base) + functions_ = base->GetAs<FunctionsT>(); + // TODO(brettw) check error and if report_error is set, do something. + } + ~EnterFunction() {} + + bool succeeded() const { return !!functions_; } + bool failed() const { return !functions_; } + + FunctionsT* functions() { return functions_; } + + private: + FunctionsT* functions_; + + DISALLOW_COPY_AND_ASSIGN(EnterFunction); +}; + +template<typename ResourceT> +class EnterResource { + public: + EnterResource(PP_Resource resource, bool report_error) + : object_(NULL) { + shared_impl::ResourceObjectBase* base = + shared_impl::TrackerBase::Get()->GetResourceAPI(resource); + if (base) + object_ = base->GetAs<ResourceT>(); + // TODO(brettw) check error and if report_error is set, do something. + } + ~EnterResource() {} + + bool succeeded() const { return !!object_; } + bool failed() const { return !object_; } + + ResourceT* object() { return object_; } + + private: + ResourceT* object_; + + DISALLOW_COPY_AND_ASSIGN(EnterResource); +}; + +} // namespace thunk +} // namespace ppapi + +#endif // PPAPI_THUNK_ENTER_H_ |