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

Disable range request loading for safari. #3315

Merged
merged 1 commit into from
Jun 3, 2013

Conversation

brendandahl
Copy link
Contributor

Fixes #3260 and #3262

@brendandahl
Copy link
Contributor Author

/botio test

@pdfjsbot
Copy link

From: Bot.io (Linux)


Received

Command cmd_test from @brendandahl received. Current queue size: 1

Live output at: http://107.21.233.14:8877/1b5c16d9c2057d3/output.txt

@pdfjsbot
Copy link

From: Bot.io (Windows)


Received

Command cmd_test from @brendandahl received. Current queue size: 1

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

@pdfjsbot
Copy link

From: Bot.io (Linux)


Success

Full output at http://107.21.233.14:8877/1b5c16d9c2057d3/output.txt

Total script time: 22.30 mins

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

@pdfjsbot
Copy link

From: Bot.io (Windows)


Success

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

Total script time: 27.85 mins

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

@yurydelendik
Copy link
Contributor

Can we move it in compatibility.js (as in yurydelendik@9461458)?

@brendandahl
Copy link
Contributor Author

/botio-linux preview

@pdfjsbot
Copy link

pdfjsbot commented Jun 3, 2013

From: Bot.io (Linux)


Received

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

Live output at: http://107.21.233.14:8877/26dccc2c727449c/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Jun 3, 2013

yurydelendik added a commit that referenced this pull request Jun 3, 2013
Disable range request loading for safari.
@yurydelendik yurydelendik merged commit 2dc062e into mozilla:master Jun 3, 2013
@brendandahl brendandahl mentioned this pull request Jun 10, 2013
@yurydelendik
Copy link
Contributor

@neosketo I think it just addresses Safari 6.0.4 issue. The version 5.1.7 sounds out of date -- please update to version 6. You are welcome to submit a patch if you find what's wrong with Safari 5.

@yurydelendik
Copy link
Contributor

hmm... I can't even seem to find Safari 6.
oh, let me be more clear that I am on Windows 7, 64-bit safari. where I am experiencing this issue.

Then your statement "it still doesn't work even after said changes." is really off-topic here. It fixes only bugs for Mac OSX and iOS. Could you create a bug for that?

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

Successfully merging this pull request may close these issues.

Pdf.js not working in safari browser on MAC OSX 10.8.2, safari version 6.0.1
3 participants