Twitter's stream should the the moz-chunked-text XHR responseType

RESOLVED FIXED in 1.2

Status

Chat Core
Twitter
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: florian, Assigned: florian)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
*** Original post on bio 1501 at 2012-06-10 10:17:00 UTC ***

*** Due to BzAPI limitations, the initial description is in comment 1 ***
(Assignee)

Comment 1

5 years ago
Created attachment 8353340 [details] [diff] [review]
Patch

*** Original post on bio 1501 as attmnt 1583 at 2012-06-10 10:17:00 UTC ***

This awesome (and completely undocumented) mozilla extension to XHR was added in https://bugzilla.mozilla.org/show_bug.cgi?id=687087

My understanding of it is that when setting responseType to moz-chunked-text, the responseText and responseBody attributes become unusable, and instead one should use the response attribute which contains only the new data received since the last progress event.

This simple change should help with memory usage when a twitter account stays connected for hours.
Attachment #8353340 - Flags: review?(clokep)
Comment on attachment 8353340 [details] [diff] [review]
Patch

*** Original change on bio 1501 attmnt 1583 at 2012-06-10 18:53:23 UTC ***

Nice find. :)
Attachment #8353340 - Flags: review?(clokep) → review+
*** Original post on bio 1501 at 2012-06-10 21:03:58 UTC ***

Checked in as http://hg.instantbird.org/instantbird/rev/2749a7494bb8
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.2
You need to log in before you can comment on or make changes to this bug.