Commit Graph

14 Commits

Author SHA1 Message Date
Benjamin Smedberg
2a5f3518de 1;2cBug 548217 - Make NP_SEEK streams work properly: they shouldn't crash if NPN_RequestRead is called after all the stream data is delivered, and they should call NPP_DestroyStream when NPN_DestroyStream is called, but not before. To this end, the classes nsPluginStreamInfo and nsPluginStreamListenerPeer have been unified: the nsNPAPIPluginStreamListener holds the stream listener peer alive so that additional range requests can be made on it. In addition the nsNPAPIPluginStreamListener holds *itself* alive for NP_SEEK streams, and only calls releases itself/calls CleanUpStream when the plugin explicitly calls NPN_DestroyStream or when the associated plugin instance is being destroyed. r=josh
--HG--
extra : rebase_source : 372df117d2b0c955605f44ee0e2fa512f0ad972b
2010-02-23 21:58:27 -05:00
Benjamin Smedberg
0b629a5939 Backed out changeset f829f942873d - bug 548217 because of topcrash bug 549112 2010-02-27 17:36:54 -05:00
Benjamin Smedberg
958042217d Bug 548217 - Make NP_SEEK streams work properly: they shouldn't crash if NPN_RequestRead is called after all the stream data is delivered, and they should call NPP_DestroyStream when NPN_DestroyStream is called, but not before. To this end, the classes nsPluginStreamInfo and nsPluginStreamListenerPeer have been unified: the nsNPAPIPluginStreamListener holds the stream listener peer alive so that additional range requests can be made on it. In addition the nsNPAPIPluginStreamListener holds *itself* alive for NP_SEEK streams, and only calls releases itself/calls CleanUpStream when the plugin explicitly calls NPN_DestroyStream or when the associated plugin instance is being destroyed. r=josh/jst 2010-02-23 21:58:27 -05:00
Benjamin Smedberg
7985a1dcb1 Backed out changeset 77dc38d8196e - bug 548217 because even though this patch is correct, it exposes a bug in the OOPP code which got backed out. 2010-02-25 03:57:57 -08:00
Benjamin Smedberg
4a49a68e09 1;2cBug 548217 - Make NP_SEEK streams work properly: they shouldn't crash if NPN_RequestRead is called after all the stream data is delivered, and they should call NPP_DestroyStream when NPN_DestroyStream is called, but not before. To this end, the classes nsPluginStreamInfo and nsPluginStreamListenerPeer have been unified: the nsNPAPIPluginStreamListener holds the stream listener peer alive so that additional range requests can be made on it. In addition the nsNPAPIPluginStreamListener holds *itself* alive for NP_SEEK streams, and only calls releases itself/calls CleanUpStream when the plugin explicitly calls NPN_DestroyStream or when the associated plugin instance is being destroyed. r=josh
--HG--
extra : rebase_source : fba343acbfcaa785c7405552d42f4558424b485c
2010-02-23 21:58:27 -05:00
Michael Wu
27fd1e717b Bug 273025 - "bad logic results in potential leak xor crash based on flow" (Improve GetURL/PostURL code, v4 (2/2)) [r+sr=jst] 2009-12-27 14:26:00 -06:00
Josh Aas
514a6c5e9d Greatly reduce the number of timers firing when multiple Carbon plugin instances are active on Mac OS X. Should reduce CPU usage. Make all instances use the same (at most two) timers for idle events. b=519598 r=roc 2009-12-15 15:44:52 -05:00
Josh Aas
d069d0092b Cocoa event model: 1) fix spurious NPP_SetWindow calls 2) propagate event model immediately. b=527115 r=smichaud r=roc 2009-11-09 22:58:48 -05:00
Josh Aas
923b0e293c Gecko should use NPAPI directly, remove nsplugindefs.h abstractions. b=516661 r/sr=jst 2009-09-16 21:30:26 -04:00
Josh Aas
65e80f6eb0 Implement Cocoa NPAPI event model. b=435041 sr=roc 2009-08-26 17:29:47 -07:00
Josh Aas
4a7715939e Bug 501194 - Remove nsplugin.h. r/sr=jst 2009-06-30 01:55:05 -04:00
Josh Aas
c81455d12a Remove plugin instance peers (nsIPluginInstancePeer, nsPIPluginInstancePeer). Second and final part of bug 500889. r/sr=jst 2009-06-29 14:53:52 -04:00
Johnny Stenback
3d56357332 Fixing bug 475646. Don't call GetPeer() on plugins as it's apparently not safe to do so on all plugins. r=joshmoz@gmail.com, sr=bzbarsky@mit.edu 2009-02-24 16:28:35 -08:00
hg@mozilla.com
465265d0d4 Free the (distributed) Lizard! Automatic merge from CVS: Module mozilla: tag HG_REPO_INITIAL_IMPORT at 22 Mar 2007 10:30 PDT, 2007-03-22 10:30:00 -07:00