Commit Graph

17 Commits

Author SHA1 Message Date
Benjamin Smedberg
1dcd610135 Followup to bug 551049 - make the closing reason match that used by the in-process host 2010-03-11 14:15:48 -05: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
Chris Jones
709ae3ce2b Bug 538537: Guard BrowserStreams from being deleted from within their constructor. r=bsmedberg 2010-01-12 17:06:18 -06: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
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
16ebf43c77 Actually implement the geturl/posturl functions on the child-side plugin host. 2009-09-15 15:48:18 -04:00
Ben Turner
c6ad60ed4d Remove 'Protocol' suffix from IPDL-generated files 2009-09-09 15:59:06 -07: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