815298 - attempting to log into a persona site crashes metro

Status: RESOLVED INCOMPLETE

(Firefox for Metro Graveyard :: General, defect)

Description

[Allison Naaktgeboren :ally]

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.

Comment 1

[Asa Dotzler [:asa]]

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.

Comment 2

[Matt Brubeck (:mbrubeck)]

Re-nominating.  Any reproducible crash should at least be investigated to figure out the cause.  This is unlikely to be limited to Persona.

Comment 3

[Asa Dotzler [: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.

Comment 4

[Brian R. Bondy [:bbondy]]

I'm not opposed to Comment 3 but just wanted to add that reproducible crashes are usually very light on developer time.

Comment 5

[Scoobidiver (away)]

(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.

Comment 6

[Asa Dotzler [:asa]]

(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.

Comment 7

[Robert Kaiser]

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.

Comment 8

[Brian R. Bondy [:bbondy]]

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.

Comment 9

[Jim Mathies [:jimm]]

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.