[totem] browser-plugin: Don't check for success when stopping a stream



commit aaf9c819977eef57e5d0c3107643ee0d02f96dbe
Author: Bastien Nocera <hadess hadess net>
Date:   Sun Dec 4 16:26:58 2011 +0100

    browser-plugin: Don't check for success when stopping a stream
    
    Fixes a warning in some cases.

 browser-plugin/totemPlugin.cpp |   15 ++++-----------
 1 files changed, 4 insertions(+), 11 deletions(-)
---
diff --git a/browser-plugin/totemPlugin.cpp b/browser-plugin/totemPlugin.cpp
index ef6407e..b385830 100644
--- a/browser-plugin/totemPlugin.cpp
+++ b/browser-plugin/totemPlugin.cpp
@@ -1098,17 +1098,10 @@ totemPlugin::UnsetStream ()
 	if (!mStream)
 		return;
 
-	if (NPN_DestroyStream (mNPP,
-                               mStream,
-                               NPRES_DONE) != NPERR_NO_ERROR) {
-                  g_warning ("Couldn't destroy the stream");
-                  /* FIXME: set mStream to NULL here too? */
-                  return;
-	}
-
-	/* Calling DestroyStream should already have set this to NULL */
-	assert (!mStream); /* Should not have a stream anymore */
-	mStream = NULL; /* just to make sure */
+	NPN_DestroyStream (mNPP,
+                           mStream,
+                           NPRES_DONE);
+        mStream = NULL;
 
 #ifdef TOTEM_NARROWSPACE_PLUGIN
         if (!mNPObjects[ePluginScriptable].IsNull ()) {



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]