ylb11 / openjpeg

Automatically exported from code.google.com/p/openjpeg
Other
0 stars 0 forks source link

Image in pdf don't display #447

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Open the attached document in chrome
2. Image on page 11 is black

This issue is from 
https://code.google.com/p/chromium/issues/detail?id=437549

Original issue reported on code.google.com by bo...@foxitsoftware.com on 4 Dec 2014 at 6:20

Attachments:

GoogleCodeExporter commented 9 years ago
@bo,

All images are not decoded properly using the latest trunk (r2949).
image 1.jp2 appears as fully black. The other are "blurred", on a black 
background which isn't right.

"Blurred" issue : Number of tile part specified is not right (same issue as 
issue 254 or issue 423).

cmap box doesn't appear in the jp2 header box & is not read at all by OpenJPEG.
cmap box is "invalid" (not according to standard but according to what it is 
intended to show) :
  Red channel shall use palette mapping using palette column 0 : OK 
  Green channel shall use direct mapping : KO, should be  palette mapping using palette column 1
  Blue channel shall use direct mapping : KO, should be  palette mapping using palette column 2

With all of the above manually corrected, images are properly decoded.

I will let someone else (Antonin ?) have a look at this because I might be 
missing something regarding ISO-15444 (which ever part/amendment/...).
Another thing is the values for precedence & approx in the colr box which 
aren't what's expected (but ignored by OpenJPEG)

Original comment by m.darb...@gmail.com on 4 Dec 2014 at 8:48

Attachments:

GoogleCodeExporter commented 9 years ago
@darbois, thank you so much for the quick debugging.

Original comment by bo...@foxitsoftware.com on 4 Dec 2014 at 9:11

GoogleCodeExporter commented 9 years ago

Original comment by m.darb...@gmail.com on 5 Dec 2014 at 11:25

GoogleCodeExporter commented 9 years ago
This is the exact same issue as issue 235.
I'm not marking this one as duplicate just yet.

Original comment by m.darb...@gmail.com on 6 Dec 2014 at 10:58

GoogleCodeExporter commented 9 years ago
Here's a patch to get those images to decode properly (with patch from issue 
254 applied as well).

Not sure we wan't this in the trunk like this but it gives the idea of what's 
going on.
Solves the issue 235 as well.

Original comment by m.darb...@gmail.com on 6 Dec 2014 at 11:08

Attachments:

GoogleCodeExporter commented 9 years ago
Forgot to mention that it's been tested against the test suite with no 
regressions. Nevertheless, it shall be reviewed.

Original comment by m.darb...@gmail.com on 6 Dec 2014 at 11:09

GoogleCodeExporter commented 9 years ago
'xpdf version 3.03' does show all images clear and sharp.

winfried

Original comment by szukw...@arcor.de on 8 Dec 2014 at 4:22

GoogleCodeExporter commented 9 years ago
Is the patch in trunk now? Please let me know when I can update the code, 
thanks!

Original comment by bo...@foxitsoftware.com on 20 Dec 2014 at 12:54

GoogleCodeExporter commented 9 years ago

Original comment by mathieu.malaterre on 7 Jan 2015 at 4:43

GoogleCodeExporter commented 9 years ago
@matthieu could you clarify this bug please (eg. the title). What remains to be 
done ? Only patch from comment #5 is needed ?

Original comment by mathieu.malaterre on 7 Jan 2015 at 5:44

GoogleCodeExporter commented 9 years ago
@mathieu,

Those files are not valid jp2 files for multiple reasons (3 issues, see comment 
#2), not sure how we could modify the title in this case.

The patch from comment #5 does not correct the issue 254 which comes with its 
own patch.

Original comment by m.darb...@gmail.com on 7 Jan 2015 at 11:15