diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-29 05:12:08 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-29 05:12:08 +0000 |
| commit | b83e93279db4504796abae4c532d270cea048af8 (patch) | |
| tree | a0f2f59e0be98e047ee65a48a4526e8b0402ac19 /ppapi/cpp/instance.cc | |
| parent | 3b3301f653cbdbf368f47f296640b672de050146 (diff) | |
| download | chromium_src-b83e93279db4504796abae4c532d270cea048af8.zip chromium_src-b83e93279db4504796abae4c532d270cea048af8.tar.gz chromium_src-b83e93279db4504796abae4c532d270cea048af8.tar.bz2 | |
Remove assertion on instance destruction.
The way that we do the wrappers now with InstanceHandles means there's no danger of dangling pointers, and actually that you can't unregister yourself during instance shutdown since the PP_Instance->pp::Instance mapping has been torn down by the time the instance destructor is called.
Review URL: https://chromiumcodereview.appspot.com/9519001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124119 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/cpp/instance.cc')
| -rw-r--r-- | ppapi/cpp/instance.cc | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/ppapi/cpp/instance.cc b/ppapi/cpp/instance.cc index c9b50ff..f2a181a 100644 --- a/ppapi/cpp/instance.cc +++ b/ppapi/cpp/instance.cc @@ -42,13 +42,6 @@ Instance::Instance(PP_Instance instance) : pp_instance_(instance) { } Instance::~Instance() { - // Ensure that all per-instance objects have been removed. Generally, these - // objects should have their lifetime scoped to the instance, such as being - // instance members or even implemented by your instance sub-class directly. - // - // If they're not unregistered at this point, they will usually have a - // dangling reference to the instance, which can cause a crash later. - PP_DCHECK(interface_name_to_objects_.empty()); } bool Instance::Init(uint32_t /*argc*/, const char* /*argn*/[], |