Eliminate PDF viewer crashes (PP-1822).

ThePalaceProject · Oct 28, 2024 · 19fa287 · 19fa287
1 parent 95ea5f2
commit 19fa287
Showing 1 changed file with 18 additions and 3 deletions.
diff --git a/...iewer-pdf-pdfjs/src/main/java/org/librarysimplified/viewer/pdf/pdfjs/PdfReaderActivity.kt b/...iewer-pdf-pdfjs/src/main/java/org/librarysimplified/viewer/pdf/pdfjs/PdfReaderActivity.kt
@@ -293,9 +293,24 @@ class PdfReaderActivity : AppCompatActivity() {
   }
 
   override fun onDestroy() {
-    this.pdfServer?.stop()
-    this.pdfReaderContainer.removeAllViews()
-    this.webView.destroy()
+    try {
+      this.pdfServer?.stop()
+    } catch (e: Throwable) {
+      this.log.debug("Failed to stop PDF server: ", e)
+    }
+
+    try {
+      this.pdfReaderContainer.removeAllViews()
+    } catch (e: Throwable) {
+      this.log.debug("Failed to remove PDF views: ", e)
+    }
+
+    try {
+      this.webView.destroy()
+    } catch (e: Throwable) {
+      this.log.debug("Failed to destroy web view: ", e)
+    }
+
     super.onDestroy()
   }