Closed Bug 1317631 Opened 3 years ago Closed 3 years ago
Crash in android
.database .Cursor Index Out Of Bounds Exception: Index -1 requested, with a size of 10 at android .database .Abstract Cursor .check Position(Abstract Cursor .java)
This bug was filed from the Socorro interface and is report bp-336a7a1a-35d6-4911-a5a9-bc0112161114. ============================================================= STR: 1. Enable Activity Stream 2. Tap the "highlights" title
Introduced in Bug 1312434
Assignee: nobody → ahunt
Status: NEW → ASSIGNED
Iteration: --- → 1.9
Depends on: 1312434
Comment on attachment 8811098 [details] Bug 1317631 - Only handle clicks for highlights https://reviewboard.mozilla.org/r/93314/#review93350
Attachment #8811098 - Flags: review?(s.kaspari) → review+
Comment on attachment 8811099 [details] Bug 1317631 - Post: add more cursor position sanity checks https://reviewboard.mozilla.org/r/93316/#review93352
Attachment #8811099 - Flags: review?(s.kaspari) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/c6f5da9786e9 Only handle clicks for highlights r=sebastian https://hg.mozilla.org/integration/autoland/rev/399dea6e9200 Post: add more cursor position sanity checks r=sebastian
Looks like this should get uplifted to 52 as well? Please request Aurora approval on the patches when you get a chance.
(In reply to Ryan VanderMeulen [:RyanVM] from comment #8) > Looks like this should get uplifted to 52 as well? Please request Aurora > approval on the patches when you get a chance. This is for an incomplete and currently still disabled feature that definitely won't be shipped or enabled in 52. I.e. in this case there's no need to uplift it because no user will ever run this code (there's no harm in uplifting either - just more effort). (This is part Activity Stream on Android, which will probably ride the trains in a future release - we're still actively developing and testing on nightly. In 52 that code is protected by a build flag.) I wonder if there's some way to tag crashes that wouldn't need uplifting, would changing the Version field be enough to not make it seem like uplift is needed?
Status flags are the answer you're looking for, thanks for the info!
You need to log in before you can comment on or make changes to this bug.