Last Comment Bug 669975 - websocket ASSERTION: nsLoadGroup not thread-safe
: websocket ASSERTION: nsLoadGroup not thread-safe
Product: Core
Classification: Components
Component: Networking: WebSockets (show other bugs)
: unspecified
: x86_64 Linux
-- normal (vote)
: mozilla8
Assigned To: Patrick McManus [:mcmanus]
: Patrick McManus [:mcmanus]
Depends on:
  Show dependency treegraph
Reported: 2011-07-07 12:28 PDT by Patrick McManus [:mcmanus]
Modified: 2012-05-30 07:18 PDT (History)
9 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch v1 (1.82 KB, patch)
2011-07-07 12:30 PDT, Patrick McManus [:mcmanus]
mcmanus: review+
blizzard: approval‑mozilla‑aurora+
Details | Diff | Splinter Review

Description User image Patrick McManus [:mcmanus] 2011-07-07 12:28:52 PDT
over in bug 669819 a stack with this assertion was reported. It turns out to be a distraction from the real issue of that bug, so I am splitting them apart.

"the real cause is a regression from 537787, mLoadGroup was supposed to be moved to a base class but was instead duplicated in both the base and derived classes.

It should also be proxy released as I mentioned in comment 6, but that's apparently only relevant on a corner case that isn't being tested."




jduell says a? aurora

"The e10s patch landed in aurora, and this fixes a relatively important yet simple bug in it."
Comment 1 User image Patrick McManus [:mcmanus] 2011-07-07 12:30:06 PDT
Created attachment 544586 [details] [diff] [review]
patch v1
Comment 2 User image Christopher Blizzard (:blizzard) 2011-07-07 14:38:45 PDT
Comment on attachment 544586 [details] [diff] [review]
patch v1

Approved for Aurora.
Comment 3 User image Christopher Blizzard (:blizzard) 2011-07-07 14:39:21 PDT
Tracking for Aurora / Firefox 7 because it's a new(ish) feature.
Comment 4 User image Marco Bonardo [::mak] 2011-07-08 05:54:37 PDT
Comment 5 User image Patrick McManus [:mcmanus] 2011-07-08 07:31:49 PDT
Comment 6 User image Virgil Dicu [:virgil] [QA] 2011-08-19 08:25:28 PDT
Mozilla/5.0 (X11; Linux x86_64; rv:7.0) Gecko/20100101 Firefox/7.0

Could you provide a test case or some clear STRs in order to get this issue verified on QA side?
Comment 7 User image Patrick McManus [:mcmanus] 2011-08-19 09:55:00 PDT
the STR was a mochitest orange failure involving test_websocket_basic.html that has cleared up. comment 0 points to 669819 for the particular failure.
Comment 8 User image Anthony Hughes (:ashughes) [GFX][QA][Mentor] 2011-09-22 15:51:34 PDT
qa- as no QA fix verification needed

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