Closed
Bug 1195755
Opened 9 years ago
Closed 9 years ago
Crash when adding a contact in Firefox Hello with debug build
Categories
(Core :: Storage: IndexedDB, defect, P1)
Core
Storage: IndexedDB
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: mikedeboer, Assigned: khuey)
References
Details
Attachments
(1 file)
(deleted),
text/plain
|
Details |
STR:
- Open NightlyDebug from the console (I'm on OSX, currently)
- Open Firefox Hello by clicking the toolbar icon and make sure you're logged in with your Firefox Account
- Go to the Contacts tab inside the Hello panel and click the 'New Contact' button.
- Fill in the 'Name' and 'Email' fields (I used 'Test' and 'info@test.com' respectively)
- Hit the 'Add Contact' button...
BOOM! (see crash signature field for more info).
Reporter | ||
Comment 1•9 years ago
|
||
This is also happens when I follow these STR with a *blank* profile:
- Follow steps 1 - 3 of the STR mentioned in comment 0.
- Click the 'Use Profile Icons' button.
Comment 2•9 years ago
|
||
Crashes in mozilla::CycleCollectedJSRuntime::RunInMetastableState, and indexedDB-related. Either way you slice it, it's probably for Kyle.
Flags: needinfo?(khuey)
Comment 3•9 years ago
|
||
FWIW, I reproduced this by modifying the testcase in bug 1170484 slightly to stash the mutation event in a property 'm' on the window, and then accessing m.length in the web console afterwards.
Assignee | ||
Comment 4•9 years ago
|
||
Can someone attach a soccorro report (or gdb backtrace) so I can see the stack in a useful manner? I need a stack that goes all the way to XRE_main.
Flags: needinfo?(mdeboer)
Flags: needinfo?(khuey)
Flags: needinfo?(bobbyholley)
Comment 5•9 years ago
|
||
Flags: needinfo?(mdeboer)
Flags: needinfo?(bobbyholley)
Comment 6•9 years ago
|
||
(This was in a non-e10s window of an e10s instance. Happened for e10s too, but I figured the non-e10s case would be easier to look at)
Assignee | ||
Comment 7•9 years ago
|
||
Thanks. This is interesting, it looks like the Mac native event loop works differently than the other platforms. I may have to relax the assertion.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → khuey
Crash Signature: Assertion failure: data.mRecursionDepth > mBaseRecursionDepth, at /Users/mikedeboer/Projects/fx-team/xpcom/base/CycleCollectedJSRuntime.cpp:1141
#01: mozilla::dom::indexedDB::IDBDatabase::Transaction(mozilla::dom::StringOrStringSequence const&, mozilla::…
Assignee | ||
Comment 10•9 years ago
|
||
Since mac can execute events from outside nsThread, I just removed the assertion on mac.
Comment 11•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•