Commit Graph

20 Commits

Author SHA1 Message Date
Josh Aas
b6cda63bd4 Fix int comparison warning in BrowserStreamChild.cpp. b=560828 r=bsmedberg 2010-05-12 14:12:27 -04:00
Benjamin Smedberg
1a54d18a03 Bug 551049 part 3: delay the delivery of NPP_URLNotify until the related stream is completely delivered and destroyed, and propagate errors from NPP_Write and NPN_DestroyStream back to NPP_URLNotify r=bent 2010-03-06 16:03:05 -05:00
Benjamin Smedberg
51a11b0266 Bug 551049 (was bug 532208) part 2 - Delay delivery of NPP_DestroyStream until stream data is delivered, and make sure that data delivery doesn't re-enter, r=bent 2010-03-06 16:02:31 -05:00
Benjamin Smedberg
39df629db6 Bug 551049 (was bug 532208) part 1 - Make PBrowserStream destructor uni-directional by delivering all data asynchronously and doing two-phase (acknowledged) destruction, r=bent 2010-02-23 16:45:00 -05:00
Benjamin Smedberg
1fc1bc4a5a Revert the RPC-style race resolution from bug 532208 in order to use the asynchronous delivery patches. 2010-03-06 11:08:19 -05:00
Chris Jones
568776b2d0 Bug 532208: Defer deletion of browser streams to later tasks. r=bsmedberg 2010-02-26 13:07:56 -06:00
Benjamin Smedberg
a97d6cd2b0 Backed out changeset e4a7ea0bb90f bug 532208 - asynchronous stream delivery because write events were being dispatched in odd re-entrant states and NPP_DestroyStream was being delivered before all the stream data was delivered. 2010-02-25 03:00:56 -08:00
Benjamin Smedberg
0d781c505c Bug 532208 - Make PBrowserStream destructor uni-directional by delivering all data asynchronously and doing two-phase (acknowledged) destruction, r=bent
--HG--
extra : rebase_source : 1155cb3c0d548712b274c81e7cf597eaf091d220
2010-02-23 16:45:00 -05:00
Benjamin Smedberg
226e0ab87f Bug 540604 - silverlight doesn't work with OOPP because it doesn't set the stype outparam for NPP_NewStream, and relies on us initializing the value with NP_NORMAL, r=cjones
--HG--
extra : rebase_source : 935580b2825b335be5a0183a79806f29f6ce404f
2010-01-26 12:05:54 -05:00
Chris Jones
709ae3ce2b Bug 538537: Guard BrowserStreams from being deleted from within their constructor. r=bsmedberg 2010-01-12 17:06:18 -06:00
Benjamin Smedberg
84db3ddf86 Bug 531859 - Remove stdout spew from dom/plugins by using NSPR logging, r=cjones
--HG--
extra : transplant_source : fW%F0%06%1F%88%2C%F3l%B3%A0%F7b%D0%13%AE%D7%BD-%26
2009-12-18 17:22:51 -05:00
Chris Jones
4a0c6ae1dc bug 525342: make IPDL dtors part of the destructed actor's protocol instead of the manager's. r=bsmedberg 2009-12-03 02:16:14 -06:00
Benjamin Smedberg
3815e2f045 More TODOs in the the plugin stream code, use the NullableString mechanisms which were invented after this landed. 2009-11-25 10:34:59 -05:00
Benjamin Smedberg
f7eb5c27ab Bug 530894 - Clean up TODOs in the plugin stream remoting: if NPN_{Get,Post}URLNotify fail, clean up the actor properly. Also deal with null postdata, which currently causes significant problems for the in-process case (NPN_PostURLNotify succeeds, but NPP_URLNotify is never called). r=bent 2009-11-25 09:26:01 -05:00
Ben Turner
d8f6fca3ab Add threadsafety assertions to plugin code 2009-10-08 12:11:13 -07:00
Benjamin Smedberg
7464979f8c More plugin crash fixes: failure from NPP_Write doesn't actually destroy the stream, it just tells the browser to destroy the stream. NPP_NewStream resulting from NPN_GetURLNotify didn't have the notifyData field set correctly. Other minor logging and cleanup.
--HG--
extra : rebase_source : 932c0290a7c1779275266607d5e853d5c1994396
2009-09-23 19:00:23 -04:00
Benjamin Smedberg
3e1cc1bae2 Fix streams to work. Be consistent about AStream* for pdata/ndata. Set up pdata before actually calling the constructor. Implement functions which were stubbed out without TODO comments. This passes the stream tests!
--HG--
extra : rebase_source : fb5dbbd43a1588efad83b80760b82f40ad1eff3f
2009-09-22 17:06:00 -04:00
Benjamin Smedberg
b64d353fd6 Implement Plugin->Browser streams. In a very basic form the first plugin instantiation passes the new plugin stream tests, but subsequent instantiations don't: I think there's some shutdown/startup issue (the plugin is being unloaded, perhaps?) 2009-09-21 10:51:35 -04:00
Chris Jones
03947f0bf3 switch message handler nsresult return types to bool. appears to break test plugin, will be temporary 2009-09-17 18:09:20 -05:00
Benjamin Smedberg
09ee5262ac PPluginStream is badly named: there is going to be a PPluginStream, but it's not this!
--HG--
rename : dom/plugins/PluginStreamChild.cpp => dom/plugins/BrowserStreamChild.cpp
rename : dom/plugins/PluginStreamChild.h => dom/plugins/BrowserStreamChild.h
rename : dom/plugins/PluginStreamParent.cpp => dom/plugins/BrowserStreamParent.cpp
rename : dom/plugins/PluginStreamParent.h => dom/plugins/BrowserStreamParent.h
rename : dom/plugins/PPluginStream.ipdl => dom/plugins/PBrowserStream.ipdl
2009-09-09 10:04:09 -04:00