Last Comment Bug 718206 - spdy socket transport event bugs
: spdy socket transport event bugs
Status: RESOLVED FIXED
[spdy]
:
Product: Core
Classification: Components
Component: Networking: HTTP (show other bugs)
: 11 Branch
: x86_64 Linux
: -- normal (vote)
: mozilla12
Assigned To: Patrick McManus [:mcmanus]
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-14 10:15 PST by Patrick McManus [:mcmanus]
Modified: 2012-01-25 18:06 PST (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch 0 (11.92 KB, patch)
2012-01-14 10:33 PST, Patrick McManus [:mcmanus]
honzab.moz: review+
Details | Diff | Splinter Review

Description Patrick McManus [:mcmanus] 2012-01-14 10:15:53 PST
the current code has all manner of problems in getting the right event to the right transaction.

take the approach of pipelining and suppress most read and write events that come from the socket transport and instead generate them from the transaction at the correct time.

this gives good results in firebug.
Comment 1 Patrick McManus [:mcmanus] 2012-01-14 10:33:33 PST
Created attachment 588666 [details] [diff] [review]
patch 0
Comment 2 Honza Bambas (:mayhemer) 2012-01-25 11:18:57 PST
Comment on attachment 588666 [details] [diff] [review]
patch 0

Review of attachment 588666 [details] [diff] [review]:
-----------------------------------------------------------------

Didn't test by my self, but looks good.

r=honzab.
Comment 3 Patrick McManus [:mcmanus] 2012-01-25 12:40:15 PST
https://hg.mozilla.org/integration/mozilla-inbound/rev/56ce42bc8afe
Comment 4 Ed Morley [:emorley] 2012-01-25 18:06:34 PST
https://hg.mozilla.org/mozilla-central/rev/56ce42bc8afe

Note You need to log in before you can comment on or make changes to this bug.