Closed Bug 815298 Opened 8 years ago Closed 8 years ago

attempting to log into a persona site crashes metro

Categories

(Firefox for Metro Graveyard :: General, defect)

x86
Windows 8.1
defect
Not set
critical

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: ally, Unassigned)

References

Details

(Keywords: crash, reproducible)

test site: http://ponyplace.ajf.me/

###!!! ASSERTION: Can't create an accessible for the document!: 'parentDocAcc',
file c:/Users/A/elm/accessible/src/base/nsAccDocManager.cpp, line 370

appears in the terminal shortly before the crash.

also note: we handle the persona login popup less than gracefully in the ui. I suspect that we handle popups in general poorly.
Keywords: crash
OS: Windows 8 → Windows 8 Metro
Whiteboard: [metro-mvp?]
I don't think Persona sites are likely a v1 requirement. If we're going to worry about compatibility, I think we should be looking at top site crashers that our real world users are most likely to see, maybe from a top 1000 sites list or something like that.
Whiteboard: [metro-mvp?] → [metro-mvp-]
Keywords: crash
Whiteboard: [metro-mvp-]
Re-nominating.  Any reproducible crash should at least be investigated to figure out the cause.  This is unlikely to be limited to Persona.
Keywords: crash, reproducible
Whiteboard: [metro-mvp?]
Flags: needinfo?(asa)
I don't think this is the right way to approach crashes. We should be looking at crash data and investigating top crashes. If it's not going to be a high value fix, I don't want to spend time on it. If this turns out to be a top crasher, then we should label it as such and then this bug becomes useful because it provides a reproducible test case. But if the signature isn't high on the crash list then we shouldn't worry about it.
Flags: needinfo?(asa)
I'm not opposed to Comment 3 but just wanted to add that reproducible crashes are usually very light on developer time.
(In reply to Asa Dotzler [:asa] from comment #3)
> We should be looking at crash data and investigating top crashes.
There is no top crash notion for such a low volume of users like for B2G.
(In reply to Scoobidiver from comment #5)
> (In reply to Asa Dotzler [:asa] from comment #3)
> > We should be looking at crash data and investigating top crashes.
> There is no top crash notion for such a low volume of users like for B2G.

Scoobidiver, what is your estimate of the ADIs we need to have to be able to identify any crash as more or less bad than any other crash? We had a couple thousand on the Elm branch and I expect us to do quite a bit better now that we're on mozilla-central.
Asa, do we actually have ADI data for MetroFirefox? If so, there's probably some bugs we need to file to get them into Socorro.
Severity: normal → critical
Hardware: x86_64 → x86
How do you reproduce by the way Ally?
The link you mentioned in Comment 0 has a login prompt and has options to sign in with twitter or facebook, but I don't see an option to sign in with persona.

I tried signing into airmozilla with persona in the metro browser and it was successful.
I tried reproducing on telemetry and couldn't reproduce either. Resolving incomplete for now since we don't have a site to reproduce on. Please reopen if you see it again.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INCOMPLETE
Whiteboard: [metro-mvp?]
Blocks: 860039
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.