Actually using HandlerThread to do rendering

cesquivias · cesquivias · commit 8576474ab414 · 2016-12-30T11:42:28.000-08:00
diff --git a/android-pdf-viewer/src/main/java/com/github/barteksc/pdfviewer/PDFView.java b/android-pdf-viewer/src/main/java/com/github/barteksc/pdfviewer/PDFView.java
@@ -686,7 +686,8 @@ public void loadComplete(PdfDocument pdfDocument) {
         if (!renderingHandlerThread.isAlive()) {
             renderingHandlerThread.start();
         }
-        renderingHandler = new RenderingHandler(this, pdfiumCore, pdfDocument);
+        renderingHandler = new RenderingHandler(renderingHandlerThread.getLooper(),
+                this, pdfiumCore, pdfDocument);
 
         if (scrollHandle != null) {
             scrollHandle.setupLayout(this);
diff --git a/android-pdf-viewer/src/main/java/com/github/barteksc/pdfviewer/RenderingHandler.java b/android-pdf-viewer/src/main/java/com/github/barteksc/pdfviewer/RenderingHandler.java
@@ -20,6 +20,7 @@
 import android.graphics.Rect;
 import android.graphics.RectF;
 import android.os.Handler;
+import android.os.Looper;
 import android.os.Message;
 
 import com.github.barteksc.pdfviewer.model.PagePart;
@@ -50,7 +51,8 @@ class RenderingHandler extends Handler {
     private Matrix renderMatrix = new Matrix();
     private final Set<Integer> openedPages = new HashSet<>();
 
-    RenderingHandler(PDFView pdfView, PdfiumCore pdfiumCore, PdfDocument pdfDocument) {
+    RenderingHandler(Looper looper, PDFView pdfView, PdfiumCore pdfiumCore, PdfDocument pdfDocument) {
+        super(looper);
         this.pdfView = pdfView;
         this.pdfiumCore = pdfiumCore;
         this.pdfDocument = pdfDocument;
@@ -65,9 +67,14 @@ void addRenderingTask(int userPage, int page, float width, float height, RectF b
     @Override
     public void handleMessage(Message message) {
         RenderingTask task = (RenderingTask) message.obj;
-        PagePart part = proceed(task);
+        final PagePart part = proceed(task);
         if (part != null) {
-            pdfView.onBitmapRendered(part);
+            pdfView.post(new Runnable() {
+                @Override
+                public void run() {
+                    pdfView.onBitmapRendered(part);
+                }
+            });
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -686,7 +686,8 @@ public void loadComplete(PdfDocument pdfDocument) {`
`686`	`686`	`if (!renderingHandlerThread.isAlive()) {`
`687`	`687`	`renderingHandlerThread.start();`
`688`	`688`	`}`
`689`		`- renderingHandler = new RenderingHandler(this, pdfiumCore, pdfDocument);`
	`689`	`+ renderingHandler = new RenderingHandler(renderingHandlerThread.getLooper(),`
	`690`	`+ this, pdfiumCore, pdfDocument);`
`690`	`691`
`691`	`692`	`if (scrollHandle != null) {`
`692`	`693`	`scrollHandle.setupLayout(this);`