Fixed: Sometimes application crashes due to a native crash when we frequently load the ZIM files in the reader.

[MohitMaliFtechiz]

Fixes #4104

There were 2 issues that caused this error.

• We were previously using lifecycleScope, but we encountered an issue where the lifecycleScope remained active when navigating away from the reader fragment. Since the fragment was still in the backStack, the scope continued running and attempted to set the new ZIM file, even though the reader screen was no longer visible. When we tried to open a new ZIM file, both files were attempting to load in the reader, causing the main page to load content from both files. This happened because the previous archive object was disposed of when the new ZIM file was added, and WebView tried to load content from the disposed archive object. To address this, we switched to using coreReaderLifeCycleScope and ensured that any ongoing operations related to setting the new ZIM file are canceled if the reader screen is destroyed.
• The second issue was that we were clearing the WebView list in our onDestroyView method but not stopping WebView's internal processes. As a result, the WebViews continued calling the shouldInterceptRequest method, which loaded internal links on the page. When a new archive object was assigned to the ZimFileReader, the previous archive was disposed of, but the WebView continued trying to load content from the disposed archive, causing an error. To fix this, we now properly stop all WebView loading before setting the new ZIM file in the archive. Additionally, we ensure that when switching to other screens, unnecessary resources are not loaded when they are no longer needed.
• Added the necessary comments in code for future reference.

screen-20241122-162059.mp4

[codecov]

Codecov Report

Attention: Patch coverage is 64.28571% with 10 lines in your changes missing coverage. Please review.

Project coverage is 58.46%. Comparing base (3305a26) to head (c86c156).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
.../kiwix/kiwixmobile/core/main/CoreReaderFragment.kt	77.27%	3 Missing and 2 partials ⚠️
...ile/nav/destination/library/CopyMoveFileHandler.kt	0.00%	3 Missing ⚠️
...bile/nav/destination/reader/KiwixReaderFragment.kt	33.33%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #4105      +/-   ##
============================================
- Coverage     58.66%   58.46%   -0.21%     
+ Complexity     1549     1534      -15     
============================================
  Files           316      316              
  Lines         13330    13351      +21     
  Branches       1698     1700       +2     
============================================
- Hits           7820     7805      -15     
- Misses         4390     4413      +23     
- Partials       1120     1133      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests
• JS Bundle Analysis - Avoid shipping oversized bundles

[kelson42]

@MohitMaliFtechiz What is the status here? Could we merge and move forward with release?

[MohitMaliFtechiz]

@kelson42 The PR is ready for review and merge now.