From 237806bbc540938a87a26b53f117c82ce75025fa Mon Sep 17 00:00:00 2001 From: Svetoslav Date: Tue, 23 Sep 2014 21:19:04 -0700 Subject: Missing pages in print preview when scorolling. It is possible that two providers for the same page are assigned to two preview views despite temporariliy while rebinding. We were however releasing the provider from the view which is being bound if not for the same page which as a result was cancelling rendering of this page which is provided to another view via another provider. Hence, having an empty page. Removed the unnecessary code to release a provider when binding the view - release is happening only if the view holder is recycled. bug:17515670 Change-Id: I197438c16cfdb363f521cd1320f154da54eea5f5 --- .../PrintSpooler/src/com/android/printspooler/ui/PageAdapter.java | 4 ---- 1 file changed, 4 deletions(-) (limited to 'packages/PrintSpooler') diff --git a/packages/PrintSpooler/src/com/android/printspooler/ui/PageAdapter.java b/packages/PrintSpooler/src/com/android/printspooler/ui/PageAdapter.java index 20e774f..fbf7204 100644 --- a/packages/PrintSpooler/src/com/android/printspooler/ui/PageAdapter.java +++ b/packages/PrintSpooler/src/com/android/printspooler/ui/PageAdapter.java @@ -312,10 +312,6 @@ public final class PageAdapter extends Adapter implements + ", pageIndexInFile: " + pageIndexInFile); } - if (provider != null && provider.getPageIndex() != pageIndexInFile) { - mPageContentRepository.releasePageContentProvider(provider); - } - provider = mPageContentRepository.acquirePageContentProvider( pageIndexInFile, content); mBoundPagesInAdapter.put(position, null); -- cgit v1.1