887097 - Get rid of Gecko event sync time out

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[(inactive) Jim Chen [:jchen] [:darchons]]

Now that we have ANR reporting, we'd rather collect ANR reports than let Gecko event syncs time out, which causes crashes anyways.

Comment 1

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: Get rid of Gecko event sync time out (v1)

I think it's still worthwhile to log long waits, but only every second.

Comment 2

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 767568 [details] [diff] [review]
Get rid of Gecko event sync time out (v1)

Review of attachment 767568 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, thanks!

Comment 3

[(inactive) Jim Chen [:jchen] [:darchons]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1f7776427454

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/1f7776427454

Comment 5

[Alex Keybl [:akeybl]]

Tracking nom since this may fix bug 863313 and friends

Comment 6

[Lukas Blakk [:lsblakk] use ?needinfo]

Since this might fix a reproducible topcrash we could go ahead with an uplift for tomorrow's beta and keep an eye on the crash stats to see if it has the intended effect. I'll approve the current patch for beta uplift based on this.

Comment 7

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 767568 [details] [diff] [review]
Get rid of Gecko event sync time out (v1)

[Triage Comment]

Will flag checkin needed.  We're taking this as a speculative fix for the crash in bug 863313 and will backout if necessary before ship if we don't see the results we're hoping for.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/688b86b2ccf6
https://hg.mozilla.org/releases/mozilla-beta/rev/598ce43be9d3