Closed
Bug 971735
Opened 8 years ago
Closed 8 years ago
java.util.ConcurrentModificationException: at java.util.ArrayList$ArrayListIterator.next(ArrayList.java) in removeHistoricalRecordsForPackage()
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox28 fixed, firefox29 fixed, firefox30 fixed, b2g-v1.3 fixed)
RESOLVED
FIXED
Firefox 30
People
(Reporter: lucasr, Assigned: lucasr)
References
Details
Crash Data
Attachments
(1 file)
1.21 KB,
patch
|
wesj
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
83b34bca-ec7a-489e-9d02-b97222140211 java.util.ConcurrentModificationException at java.util.ArrayList$ArrayListIterator.next(ArrayList.java:569) at org.mozilla.gecko.widget.ActivityChooserModel.access$600(ActivityChooserModel.java:104) at org.mozilla.gecko.widget.ActivityChooserModel$DataModelPackageMonitor.onReceive(ActivityChooserModel.java:1218) at android.app.LoadedApk$ReceiverDispatcher$Args.run(LoadedApk.java:763) at android.os.Handler.handleCallback(Handler.java:725) at android.os.Handler.dispatchMessage(Handler.java:92) at android.os.Looper.loop(Looper.java:175) at android.app.ActivityThread.main(ActivityThread.java:5279) at java.lang.reflect.Method.invokeNative(Native Method) at java.lang.reflect.Method.invoke(Method.java:511) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1102) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:869) at dalvik.system.NativeStart.main(Native Method)
Assignee | ||
Comment 1•8 years ago
|
||
Assignee | ||
Comment 2•8 years ago
|
||
Comment on attachment 8374851 [details] [diff] [review] Remove element using iterator while looping in ArrayList (r=wesj) Top crasher in Fx29 and Fx30. Apparently caused by bug 881861.
Attachment #8374851 -
Flags: review?(wjohnston)
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 969743
Comment 4•8 years ago
|
||
Comment on attachment 8374851 [details] [diff] [review] Remove element using iterator while looping in ArrayList (r=wesj) Review of attachment 8374851 [details] [diff] [review]: ----------------------------------------------------------------- I think you're right. This is simpler. Lets just use it. Flipping the dup around....
Attachment #8374851 -
Flags: review?(wjohnston) → review+
Updated•8 years ago
|
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Assignee | ||
Comment 6•8 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/7273e402c9e6
Comment 7•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/7273e402c9e6
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
Comment 8•8 years ago
|
||
Comment on attachment 8374851 [details] [diff] [review] Remove element using iterator while looping in ArrayList (r=wesj) [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 881861, which I'm also noming for uplift User impact if declined: Crash Testing completed (on m-c, etc.): Landed on mc last week. Risk to taking this patch (and alternatives if risky): This is pretty simple and very low risk. Alternatively here is to not uplift 881861 String or IDL/UUID changes made by this patch: none.
Attachment #8374851 -
Flags: approval-mozilla-beta?
Attachment #8374851 -
Flags: approval-mozilla-aurora?
Updated•8 years ago
|
Attachment #8374851 -
Flags: approval-mozilla-beta?
Attachment #8374851 -
Flags: approval-mozilla-beta+
Attachment #8374851 -
Flags: approval-mozilla-aurora?
Attachment #8374851 -
Flags: approval-mozilla-aurora+
Updated•8 years ago
|
status-firefox29:
--- → affected
status-firefox30:
--- → fixed
Updated•8 years ago
|
status-firefox28:
--- → fixed
Updated•8 years ago
|
status-b2g-v1.3:
--- → fixed
Updated•1 year ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•