Closed
Bug 244571
Opened 21 years ago
Closed 9 years ago
Support for multiple open sockets to push/pull files from Roaming store
Categories
(Core Graveyard :: Profile: Roaming, enhancement)
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: jmccabe, Unassigned)
Details
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a2) Gecko/20040524
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a2) Gecko/20040524
Currently the Roaming Profile transfers files synchronously (start transfer for
file foo, wait for completion, begin transfer for file bar, wait for completion,
etc).
It would be a huge boon to have Moz open multiple connections to the server.
Reproducible: Always
Steps to Reproduce:
Comment 1•20 years ago
|
||
> It would be a huge boon to have Moz open multiple connections to the server.
Why? Esp. if you're bandwidth-limited (as you probably are with servers on the
Internet, which you said you use), it shouldn't make much of a difference, does it?
There is explicit code to transfer in parallel, see
<http://lxr.mozilla.org/seamonkey/source/extensions/sroaming/resources/content/transfer/transfer.js#61>
(Comment at the top)
<http://lxr.mozilla.org/seamonkey/source/extensions/sroaming/src/Stream.cpp#218>
(Comment and |ioParamBlock->SetInt(1, 2);| in Stream::DownUpLoad())
I just didn't think that there is a good and frequent reason to do that, so I
didn't implement a way to set this setting (in the Mozilla registry), but
hardcoded it. If you wanted to implement this, you just need to change
Stream.cpp and add a registry value like the existing |SavePassword|. I don't
see enough use for it to have UI, though, so I don't know how you'd change the
Mozilla registry as a user.
If you want to hack it in your copy of Mozilla, open transfer.js in sroaming.jar
and change
this.serial = serial;
(currently like 99) to
this.serial = false;
Of course, any warranty is void, if you remove the seal like this ;-).
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•20 years ago
|
||
> ioParamBlock->SetInt(1, 2);
Actually, it seems there is a bug somewhere. 2 (which I use here) is defined as
parallel, not serial, but it seems we do transfer serially, not in parallel.
(Yeah, magic numbers :-(. Because of nsIDialogParamBlock.)
Assignee: nobody → ben.bucksch
Component: Profile: BackEnd → Profile: Roaming
QA Contact: core.profile-manager-backend → core.profile-roaming
Updated•20 years ago
|
Assignee: ben.bucksch → nobody
Blocks: 1243899
Comment 3•9 years ago
|
||
This bug is filed in a bugzilla component related to pre-Firefox code which no longer exists. I believe it is no longer relevant and I am therefore closing it INCOMPLETE.
If you believe that this bug is still valid and needs to be fixed, please reopen it and move it to the Toolkit:Startup and Profile System product/component.
Assignee | ||
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•