Browse Source

Address review comments by yury

Julian Viereck 14 years ago
parent
commit
af8e2a1189
  1. 9
      src/core.js
  2. 8
      src/image.js
  3. 2
      src/obj.js

9
src/core.js

@ -561,14 +561,7 @@ var PDFDoc = (function pdfDoc() { @@ -561,14 +561,7 @@ var PDFDoc = (function pdfDoc() {
switch (type) {
case 'JpegStream':
var imageData = data[2];
var src = 'data:image/jpeg;base64,' + window.btoa(imageData);
var img = new Image();
img.onload = (function jpegImageLoaderOnload() {
this.objs.resolve(id, img);
}).bind(this);
img.src = src;
loadJpegStream(id, imageData, this.objs);
break;
case 'Font':
var name = data[2];

8
src/image.js

@ -228,3 +228,11 @@ var PDFImage = (function pdfImage() { @@ -228,3 +228,11 @@ var PDFImage = (function pdfImage() {
};
return constructor;
})();
function loadJpegStream(id, imageData, objs) {
var img = new Image();
img.onload = (function jpegImageLoaderOnload() {
objs.resolve(id, img);
});
img.src = 'data:image/jpeg;base64,' + window.btoa(imageData);
}

2
src/obj.js

@ -589,7 +589,7 @@ var XRef = (function xRefXRef() { @@ -589,7 +589,7 @@ var XRef = (function xRefXRef() {
e = parser.getObj();
}
// Don't cache streams since they are mutable (except images).
if (!isStream(e) || e.src)
if (!isStream(e) || e instanceof JpegStream)
this.cache[num] = e;
return e;
}

Loading…
Cancel
Save