Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PDFThumbnailView] Re-factor the canvas to image conversion such that we always render to a canvas, and then replace it with an image once rendering is done #7029

Merged
merged 1 commit into from
Mar 7, 2016

Conversation

Snuffleupagus
Copy link
Collaborator

This is a follow-up to PRs #6299 and #6441.

The patch also adds an option to PDFThumbnailView, that disables the canvas-to-image conversion entirely, which might be useful in the context of issue #7026.

@Snuffleupagus Snuffleupagus changed the title [PDFThumnailView] Re-factor the canvas to image conversion such that we always render to a canvas, and then replace it with an image once rendering is done [PDFThumbnailView] Re-factor the canvas to image conversion such that we always render to a canvas, and then replace it with an image once rendering is done Feb 25, 2016
…that we always render to a `canvas`, and then replace it with an `image` once rendering is done

*This is a follow-up to PRs 6299 and 6441.*

The patch also adds an option to `PDFThumbnailView`, that disables the canvas-to-image conversion entirely, which might be useful in the context of issue 7026.
@Snuffleupagus
Copy link
Collaborator Author

/botio-linux preview

@pdfjsbot
Copy link

From: Bot.io (Linux)


Received

Command cmd_preview from @Snuffleupagus received. Current queue size: 0

Live output at: http://107.21.233.14:8877/cae578a0bbdb086/output.txt

yurydelendik added a commit that referenced this pull request Mar 7, 2016
…canvas-to-image

[PDFThumbnailView] Re-factor the `canvas` to `image` conversion such that we always render to a `canvas`, and then replace it with an `image` once rendering is done
@yurydelendik yurydelendik merged commit 4065a21 into mozilla:master Mar 7, 2016
@yurydelendik
Copy link
Contributor

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants