ICEpdf
  1. ICEpdf
  2. PDF-711

Color space colour or pattern paint issue

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 6.0
          • Fix Version/s: 6.0
          • Component/s: Core/Parsing, Core/Rendering
          • Labels:
            None
          • Environment:
            any

            Description

            This issue came in through support. The PDF displays a fairly complex architecture plan and has server section that are appearing black instead of light grey. The issue is likely related to ColorSpace issue or a pattern paint issue where tiling image is not being correctly decoded.

            Further investigation is needed to isolate the reason for the paint issue.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Corless created issue -
              Arran Mccullough made changes -
              Field Original Value New Value
              Salesforce Case Reference 5007000000Yd2I0AAJ
              Patrick Corless made changes -
              Fix Version/s 5.1 [ 10675 ]
              Patrick Corless made changes -
              Affects Version/s 5.2 [ 10970 ]
              Affects Version/s 5.0.4 [ 11072 ]
              Patrick Corless made changes -
              Fix Version/s 5.2 [ 10970 ]
              Fix Version/s 5.1 [ 10675 ]
              Arran Mccullough made changes -
              Support Case References Support Case #12588 - https://icesoft.my.salesforce.com/5007000000Yd2I0
              Support Case #12667 - https://icesoft.my.salesforce.com/5007000000Zwk2M
              Patrick Corless made changes -
              Component/s Core/Rendering [ 10974 ]
              Patrick Corless made changes -
              Fix Version/s 5.3 [ 12070 ]
              Fix Version/s 5.2 [ 10970 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #44470 Fri Apr 10 09:31:17 MDT 2015 patrick.corless PDF-711 fixed graphic state save issue when processsing forms.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/GraphicsState.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/content/AbstractContentParser.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #44486 Mon Apr 13 13:25:58 MDT 2015 patrick.corless PDF-711 fixed copy issue in graphic state saving where the alpha value of a colour could be lost.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/GraphicsState.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Hide
              Permalink
              Patrick Corless added a comment -

              This took a while to isolate given the 43,000 draw operations. I got lucky though and the colour issue can be reproduced in the first 713 bytes of content stream. It turns out that Separation colour space uses a tint value which we translate to the transparency for rgba. However in our Graphics state we copy colours using only the rgb values and ignore the alpha and thus black is being drawn.

              Touched up the code and made sure we aren't doing the bad copy anywhere else. Also made a change to Separation colour space where we were changing the named colour value which I don't believe is correct.

              Show
              Patrick Corless added a comment - This took a while to isolate given the 43,000 draw operations. I got lucky though and the colour issue can be reproduced in the first 713 bytes of content stream. It turns out that Separation colour space uses a tint value which we translate to the transparency for rgba. However in our Graphics state we copy colours using only the rgb values and ignore the alpha and thus black is being drawn. Touched up the code and made sure we aren't doing the bad copy anywhere else. Also made a change to Separation colour space where we were changing the named colour value which I don't believe is correct.
              Patrick Corless made changes -
              Fix Version/s 5.2 [ 10970 ]
              Fix Version/s 5.3 [ 12070 ]
              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 #44555 Thu Apr 23 07:39:08 MDT 2015 patrick.corless PDF-711 fixed copy issue in graphic state saving where the alpha value of a colour could be lost.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/util/content/AbstractContentParser.java
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/GraphicsState.java
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #44563 Thu Apr 23 11:28:26 MDT 2015 patrick.corless PDF-711 fixed graphic state save regression.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/content/AbstractContentParser.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #44564 Thu Apr 23 11:30:32 MDT 2015 patrick.corless PDF-711 fixed graphic state save regression.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/util/content/AbstractContentParser.java
              Hide
              Permalink
              Patrick Corless added a comment -

              Marking as closed

              Show
              Patrick Corless added a comment - Marking as closed
              Patrick Corless made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: