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.

Sign up for GitHub

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

Jump to bottom

Fixes CCITTFaxStream regression #5149

Merged
merged 1 commit into from
Aug 7, 2014
Merged

Fixes CCITTFaxStream regression #5149

merged 1 commit into from
Aug 7, 2014

Conversation

timvandermeij
Copy link
Contributor

Fixes #5144.

@timvandermeij
Copy link
Contributor Author

/botio test

@pdfjsbot
Copy link

pdfjsbot commented Aug 7, 2014

From: Bot.io (Linux)

Received

Command cmd_test from @timvandermeij received. Current queue size: 0

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

@pdfjsbot
Copy link

pdfjsbot commented Aug 7, 2014

From: Bot.io (Windows)

Received

Command cmd_test from @timvandermeij received. Current queue size: 0

Live output at: http://107.22.172.223:8877/dc4ac57ed2a699b/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Aug 7, 2014

From: Bot.io (Windows)

Success

Full output at http://107.22.172.223:8877/dc4ac57ed2a699b/output.txt

Total script time: 19.83 mins

  • Font tests: Passed
  • Unit tests: Passed
  • Regression tests: Passed

@pdfjsbot
Copy link

pdfjsbot commented Aug 7, 2014

From: Bot.io (Linux)

Success

Full output at http://107.21.233.14:8877/ac17e7268d26c18/output.txt

Total script time: 22.57 mins

  • Font tests: Passed
  • Unit tests: Passed
  • Regression tests: Passed

@yurydelendik
Copy link
Contributor

There is some copyrighted material in the test pdf.

@timvandermeij
Copy link
Contributor Author

Since the OP posted a Speedyshare link to it, we also cannot turn it into a link in the test manifest (Speedyshare links are temporary), so I'll remove the test alltogether then. A bit unfortunate, but there is no other alternative I think.

@timvandermeij
Copy link
Contributor Author

/botio-linux preview

@pdfjsbot
Copy link

pdfjsbot commented Aug 7, 2014

From: Bot.io (Linux)

Received

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

Live output at: http://107.21.233.14:8877/39dc126374cc982/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Aug 7, 2014

From: Bot.io (Linux)

Success

Full output at http://107.21.233.14:8877/39dc126374cc982/output.txt

Total script time: 0.75 mins

Published

@timvandermeij
Copy link
Contributor Author

Only the test has been removed. No other changes were made.

yurydelendik added a commit that referenced this pull request Aug 7, 2014
@yurydelendik
Merge pull request #5149 from timvandermeij/ccitt-regression
3dce05f 
Fixes CCITTFaxStream regression
@yurydelendik yurydelendik merged commit 3dce05f into mozilla:master Aug 7, 2014
@yurydelendik
Copy link
Contributor

Thanks

@timvandermeij timvandermeij deleted the ccitt-regression branch August 7, 2014 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Document stuck loading
3 participants
@timvandermeij @pdfjsbot @yurydelendik