Merge \"Syncronize renderPage vs. onServiceDisconnected\" into nyc-mr1-dev

am: c7c647bbf0 Change-Id: Ibd18df2822dbb0993eb4fc22392c3c523197c540
2016-06-28 16:18:09 +00:00
parent c8fea44e5e c7c647bbf0
commit c76f44fab4
1 changed files with 9 additions and 3 deletions
--- a/packages/PrintSpooler/src/com/android/printspooler/model/PageContentRepository.java
+++ b/packages/PrintSpooler/src/com/android/printspooler/model/PageContentRepository.java
@@ -838,9 +838,15 @@ public final class PageContentRepository {

                    try (ParcelFileDescriptor source = pipe[0]) {
                        try (ParcelFileDescriptor destination = pipe[1]) {
-
-                            mRenderer.renderPage(mPageIndex, bitmap.getWidth(), bitmap.getHeight(),
-                                    mRenderSpec.printAttributes, destination);
+                            synchronized (mLock) {
+                                if (mRenderer != null) {
+                                    mRenderer.renderPage(mPageIndex, bitmap.getWidth(),
+                                            bitmap.getHeight(), mRenderSpec.printAttributes,
+                                            destination);
+                                } else {
+                                    throw new IllegalStateException("Renderer is disconnected");
+                                }
+                            }
                        }

                        BitmapSerializeUtils.readBitmapPixels(bitmap, source);