Closed Bug 560591 Opened 13 years ago Closed 13 years ago

Increase necko buffer sizes for fennec

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: dougt, Assigned: dougt)

Details

Attachments

(1 file)

16k x 64count patch
613 bytes, patch
mfinkle
: review+
Details | Diff | Splinter Review 
Since bug 545869 has landed, we should used the data that mfinkle provided increase our necko buffer sizes for Fennec.

Currently the buffer sizes are set in all.js.
Attachment #440276 - Attachment is patch: true
Attachment #440276 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 440276 [details] [diff] [review]
16k x 64count patch


>+// See bug 545869 for details on why these are set the way they are
>+pref("network.buffer.cache.count", 64);
>+pref("network.buffer.cache.size",  16384);

Can you put these just below the other "network." preferences?
Attachment #440276 - Flags: review?
(In reply to comment #1)
> (From update of attachment 440276 [details] [diff] [review])
> 
> >+// See bug 545869 for details on why these are set the way they are
> >+pref("network.buffer.cache.count", 64);
> >+pref("network.buffer.cache.size",  16384);
> 
> Can you put these just below the other "network." preferences?

Also, my data was collected with "network.buffer.cache.count" = 24
Comment on attachment 440276 [details] [diff] [review]
16k x 64count patch

Meant to r+ this
Attachment #440276 - Flags: review? → review+
This was fixed a while ago:
http://hg.mozilla.org/mobile-browser/rev/dd6d03cbc981
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee: nobody → doug.turner
You need to log in before you can comment on or make changes to this bug.