ICEpdf
  1. ICEpdf
  2. PDF-1152

Images fails to load on quick zoom in zoom out cycles.

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 6.2.3
          • Fix Version/s: 6.2.4
          • Component/s: Core/Parsing, Core/Rendering
          • Labels:
            None
          • Environment:
            Viewer RI

            Description

            When testing the Viewer RI it's possible to very quick zoom in/out with the ctrl key and the mouse wheel. Occassionaly an image will fail to load and won't be painted until some indeterminate amount of time passes.

            While working on a ImageReference type I decide to dig a little deeper. When a zoom in/out occurs before the previous paint thtread finishes we interrupt the old thread and start painting fresh on a new painter thread. The interrupt exception is picked up the the CacheImageReference implementation and it returns a null image value. Becuase of the cached nature the image will not be loaded again until the parent page view clears it cache. The image will eventually retrieved at later time when all the conditions line up.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Corless created issue -
              Patrick Corless made changes -
              Field Original Value New Value
              Fix Version/s 6.2.4 [ 13099 ]
              Hide
              Permalink
              Patrick Corless added a comment -

              I've change the ImageReference class to now catch, log and throw the interupt exception so it publes up to the call page paint. This allows to properly flush the prevous buffer and start painting and initializing in a clean state.

              Show
              Patrick Corless added a comment - I've change the ImageReference class to now catch, log and throw the interupt exception so it publes up to the call page paint. This allows to properly flush the prevous buffer and start painting and initializing in a clean state.
              Hide
              Permalink
              Patrick Corless added a comment -

              Marking as fixed.

              Show
              Patrick Corless added a comment - Marking as fixed.
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51507 Wed May 10 15:02:05 MDT 2017 patrick.corless PDF-1152 fixed a concurrency issue which prevented interrupted images
              from reinitializing in a timely fashion.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/commands/ImageDrawCmd.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/MipMappedImageReference.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/InlineImageStreamReference.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/ScaledImageReference.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/ImageReference.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/CachedImageReference.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51508 Wed May 10 15:03:43 MDT 2017 patrick.corless PDF-1152 fixed a concurrency issue which prevented interrupted images
              from reinitializing in a timely fashion.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/CachedImageReference.java
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/commands/ImageDrawCmd.java
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/ScaledImageReference.java
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/ImageReference.java
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/InlineImageStreamReference.java
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/MipMappedImageReference.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51511 Wed May 10 15:10:08 MDT 2017 patrick.corless PDF-1152 fixed a concurrency issue which prevented interrupted images
              from reinitializing in a timely fashion.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/Shapes.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51512 Wed May 10 15:10:39 MDT 2017 patrick.corless PDF-1152 fixed a concurrency issue which prevented interrupted images
              from reinitializing in a timely fashion.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/Shapes.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51517 Thu May 11 15:30:33 MDT 2017 patrick.corless PDF-1152 backing out font substitution names changes.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/fonts/FontManager.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51521 Fri May 12 11:28:04 MDT 2017 patrick.corless PDF-1152 further works to image concurrency, remove synchronized call
              on getImage as we have all the data at this point and should be thread
              safe.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/ImageStream.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51526 Mon May 15 15:12:17 MDT 2017 patrick.corless PDF-1152 lowered logging level.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/TilingPattern.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51527 Mon May 15 15:12:28 MDT 2017 patrick.corless PDF-1152 lowered logging level.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/TilingPattern.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51540 Wed May 17 13:45:55 MDT 2017 patrick.corless PDF-1152 further works to image concurrency, remove synchronized call
              on getImage as we have all the data at this point and should be thread
              safe.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/ImageStream.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51541 Wed May 17 13:47:38 MDT 2017 patrick.corless PDF-1152 backing out font substitution names changes.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/fonts/FontManager.java
              Patrick Corless made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Patrick Corless
                  Reporter:
                  Patrick Corless
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: