From 55801ac3a9d6ea284fa27350ffa59e39197f3aca Mon Sep 17 00:00:00 2001 From: "brettw@chromium.org" Date: Mon, 4 Apr 2011 17:47:17 +0000 Subject: Remove assertion when destroying a plugin that doesn't exist. The DidDestroy function will always be called, so the proxy should always add it regardless of the initialization status. I also did some clarification of the Destroy path. BUG=78112 Review URL: http://codereview.chromium.org/6706025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80336 0039d316-1c4b-4281-b951-d872f2087c98 --- ppapi/proxy/ppp_instance_proxy.cc | 5 ----- 1 file changed, 5 deletions(-) (limited to 'ppapi/proxy/ppp_instance_proxy.cc') diff --git a/ppapi/proxy/ppp_instance_proxy.cc b/ppapi/proxy/ppp_instance_proxy.cc index bb0ad79..393f031 100644 --- a/ppapi/proxy/ppp_instance_proxy.cc +++ b/ppapi/proxy/ppp_instance_proxy.cc @@ -186,11 +186,6 @@ void PPP_Instance_Proxy::OnMsgDidCreate( *result = ppp_instance_target()->DidCreate(instance, static_cast(argn.size()), &argn_array[0], &argv_array[0]); - if (!*result) { - // In the failure to create case, this plugin instance will be torn down - // without further notification, so we also need to undo the routing. - plugin_dispatcher->DidDestroyInstance(instance); - } } void PPP_Instance_Proxy::OnMsgDidDestroy(PP_Instance instance) { -- cgit v1.1