812848 - [unagi] parent process is crash for opening & closing camera app repeatedly

Status: RESOLVED FIXED

(Core :: DOM: Geolocation, defect)

Description

[Thinker Li [:sinker]]

Attachment: Fix nsGeolocation::ClearWatch()

This is a bug of geolocation, weird!!!

STR:
 - launch camera app
 - press home button to get the card view
 - swipe camera app
 - repeat above steps for several times

Comment 1

[Jason Smith [:jsmith]]

So what exactly is the expected vs. actual results for this bug?

Comment 2

[Jason Smith [:jsmith]]

(In reply to Jason Smith [:jsmith] from comment #1)
> So what exactly is the expected vs. actual results for this bug?

Oh wait - read the title - it's a crash. My bad.

Comment 3

[Robert Kaiser]

Do we have crash reports that were sent for this? I'd like to understand which crash signatures in submitted data are understood and which need to be filed, so it would be helpful to know which one this is.

Comment 4

[Doug Turner (:dougt)]

we should uplift

Comment 5

[Mozilla RelEng Bot]

Try run for 1773185a1643 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=1773185a1643
Results (out of 2 total builds):
    success: 1
    failure: 1
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-1773185a1643

Comment 6

[Mozilla RelEng Bot]

Try run for 1773185a1643 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=1773185a1643
Results (out of 5 total builds):
    exception: 2
    success: 2
    failure: 1
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-1773185a1643

Comment 7

[Thinker Li [:sinker]]

https://tbpl.mozilla.org/?tree=Try&rev=9d44c4cf8713

Comment 8

[Thinker Li [:sinker]]

Attachment: fix typo of code of geolocation

r=dougt

Comment 9

[Daniel Holbert [:dholbert]]

Landed: https://hg.mozilla.org/integration/mozilla-inbound/rev/df977f73e631

@thinker: For future reference, the "checkin+" patch flag (when it's used, which is infrequently) indicates "this has already been checked in".  So -- you probably didn't want to set it, back in comment 8.

If you're asking for a specific patch to be checked in, use "checkin?".  (But in cases where there's just one patch, you can just ignore that flag and rely on "checkin-needed".)


Any chance we can add an automated test for this?

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/cea5c31daab9
(This was an empty cset due to me push racing with dholbert)

Comment 11

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/df977f73e631
https://hg.mozilla.org/mozilla-central/rev/cea5c31daab9

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/3e184d0a1984
https://hg.mozilla.org/releases/mozilla-beta/rev/b5abc0422693