Last Comment Bug 753845 - Race condition in selectTab causes all manner of badness
: Race condition in selectTab causes all manner of badness
Product: Firefox for Android
Classification: Client Software
Component: General (show other bugs)
: unspecified
: All Android
: -- normal (vote)
: Firefox 15
Assigned To: away[Nov24,Dec5) Kartikaya Gupta (
: Sebastian Kaspari (:sebastian)
Depends on:
Blocks: 745250
  Show dependency treegraph
Reported: 2012-05-10 09:47 PDT by away[Nov24,Dec5) Kartikaya Gupta (
Modified: 2012-05-21 02:35 PDT (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch (2.05 KB, patch)
2012-05-10 09:47 PDT, away[Nov24,Dec5) Kartikaya Gupta (
margaret.leibovic: review+
mark.finkle: approval‑mozilla‑aurora+
Details | Diff | Splinter Review

Description away[Nov24,Dec5) Kartikaya Gupta ( 2012-05-10 09:47:13 PDT
Created attachment 622763 [details] [diff] [review]

If the runnable in selectTab() starts running before selectTab() finishes, isSelectedTab(tab) fails and a bunch of Important Things (TM) don't happen.

Nom'ing for betaN+ blocker since this is probably the root cause for a bunch of bugs I've seen go by (things like titlebar not updating and such).
Comment 1 :Margaret Leibovic 2012-05-10 10:39:33 PDT
Comment on attachment 622763 [details] [diff] [review]

Good catch.
Comment 2 away[Nov24,Dec5) Kartikaya Gupta ( 2012-05-10 10:43:18 PDT
Comment 3 away[Nov24,Dec5) Kartikaya Gupta ( 2012-05-10 10:44:22 PDT
Comment on attachment 622763 [details] [diff] [review]

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: randomly things won't update on switching tabs
Testing completed (on m-c, etc.): locally
Risk to taking this patch (and alternatives if risky): mobile-only
String changes made by this patch: none
Comment 4 Joe Drew (not getting mail) 2012-05-10 18:33:58 PDT
Comment 5 away[Nov24,Dec5) Kartikaya Gupta ( 2012-05-11 14:26:17 PDT

Note You need to log in before you can comment on or make changes to this bug.