Skip to content
Snippets Groups Projects

Fix assertion caused by system-initiated process death

Merged Daniel Lublin requested to merge 2109-assertion-in-rssfeedviewmodel into master

Closes #2109 (closed)

Here is how to provoke the crash/assertion on installations that is not running with this fix merged into the code:

  1. First navigate to android Settings -> System - Developer options and enable the "Don't keep activities" option (under heading APPS). (I guess the exact location/wording can vary between devices).
  2. Start Briar
  3. Navigate to Blogs page and tap RSS Feeds (in overflow menu).
  4. Plus + to import new feed
  5. Type URL https://example.com
  6. Tap Import
  7. Wait for error dialog.
  8. Tap android home button, (may vary), to show the home screen with all the apps.
  9. (the android system will now "not keep" the rss feed import activity, stopping it that is)
  10. Tap the "app overview" button, (may vary) and navigate (tap) back to Briar.
  11. Tap Try Again
  12. Enjoy Briad crashed icon.

With the fix, we should obviously not have a crash. Enabling android to not keep activities is the way to quickly provoking this crash, so it is helpful when verifying the fix.

Edited by Daniel Lublin

Merge request reports

Merge request pipeline #7116 passed

Merge request pipeline passed for e2879cd6

Merged by Torsten GroteTorsten Grote 3 years ago (Jul 20, 2021 1:03pm UTC)

Loading

Pipeline #7147 failed

Pipeline failed for d1c0f1b2 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply