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

Use Opaque Canvas #4556

Merged
merged 1 commit into from
Apr 10, 2014
Merged

Use Opaque Canvas #4556

merged 1 commit into from
Apr 10, 2014

Conversation

p01
Copy link
Contributor

@p01 p01 commented Apr 3, 2014

This improves painting performance and text rendering quality in some browsers, without any measurable performance impact.

Scrolling through the TraceMonkey PDF, several times, at three different zoom level with and without opaque Canvas, the average time to draw a page was the same in both Firefox Aurora and Opera Developer.

@timvandermeij
Copy link
Contributor

/botio-linux preview

@pdfjsbot
Copy link

pdfjsbot commented Apr 3, 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/16bac150d351134/output.txt

@timvandermeij
Copy link
Contributor

/botio test

@pdfjsbot
Copy link

pdfjsbot commented Apr 3, 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/752af7e2d4c355f/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Apr 3, 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/d0c63ba7f63fb5b/output.txt

@pdfjsbot
Copy link

pdfjsbot commented Apr 3, 2014

From: Bot.io (Windows)


Failed

Full output at http://107.22.172.223:8877/752af7e2d4c355f/output.txt

Total script time: 21.88 mins

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

Image differences available at: http://107.22.172.223:8877/752af7e2d4c355f/reftest-analyzer.html#web=eq.log

@pdfjsbot
Copy link

pdfjsbot commented Apr 3, 2014

From: Bot.io (Linux)


Failed

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

Total script time: 25.68 mins

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

Image differences available at: http://107.21.233.14:8877/d0c63ba7f63fb5b/reftest-analyzer.html#web=eq.log

@timvandermeij
Copy link
Contributor

Hm, this really seems to break a lot. Any ideas?

@brendandahl
Copy link
Contributor

@timvandermeij
Copy link
Contributor

@p01 Could you squash these commits into one?

@p01
Copy link
Contributor Author

p01 commented Apr 10, 2014

Done.
Transparent canvases are still used for the CachedCanvases and in the "JpegDecode" message in api.js

Every PDF I checked from the previous test run passes now.

@Snuffleupagus
Copy link
Collaborator

/botio test

@pdfjsbot
Copy link

From: Bot.io (Linux)


Received

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

Live output at: http://107.21.233.14:8877/62820354091e7a9/output.txt

@pdfjsbot
Copy link

From: Bot.io (Windows)


Received

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

Live output at: http://107.22.172.223:8877/19054be7e059b19/output.txt

@pdfjsbot
Copy link

From: Bot.io (Windows)


Success

Full output at http://107.22.172.223:8877/19054be7e059b19/output.txt

Total script time: 21.32 mins

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

@pdfjsbot
Copy link

From: Bot.io (Linux)


Success

Full output at http://107.21.233.14:8877/62820354091e7a9/output.txt

Total script time: 25.38 mins

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

@yurydelendik
Copy link
Contributor

/botio-linux preview

@pdfjsbot
Copy link

From: Bot.io (Linux)


Received

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

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

yurydelendik added a commit that referenced this pull request Apr 10, 2014
@yurydelendik yurydelendik merged commit d61ee23 into mozilla:master Apr 10, 2014
@yurydelendik
Copy link
Contributor

Thanks

yurydelendik added a commit that referenced this pull request Apr 11, 2014
@yurydelendik
Copy link
Contributor

Reverted the patch by f6c631b due to heavy while->black->white flickering on Chrome browser. Needs some work.

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.

6 participants