Created attachment 8426804
Implement size for recording surface created for windows printing

Ok, confirmed that it's still broken on beta.

It appears that for windows printing we create a gfxWindowsSurface that
is backed by a cairo recording surface. So when we call
CreateSimilarSurface, we end up with a gfxUnknownSurface (since we don't
have a c++ class for recording surfaces).

GetSize() isn't implemented for gfxUnknownSurface and we hit the same
bug as before.

This patch seems like the most unobtrusive way to fix this, especially
since we're pushing towards using Moz2D directly everywhere.

I guess the *right* fix would be to add gfxRecordingSurface, and to
expose new cairo API to obtain the size from a
cairo_recording_surface_t. I don't think that's worth doing for code we
want to get rid of.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to firefox in Ubuntu.
https://bugs.launchpad.net/bugs/1315020

Title:
  PDF.js prints blank pages

Status in The Mozilla Firefox Browser:
  Fix Released
Status in “firefox” package in Ubuntu:
  Confirmed

Bug description:
  The recent upgrade to 29.0+build1-0ubuntu0.12.04.2 (on Ubuntu 12.04.4
  LTS) cause printing from PDF.js to emit essentially blank output (the
  spooled files are very small, and contain showpage directives with no
  page content, i.e., blank pages come out of the printer).  Users can
  download the file from the down-arrow menu and print using a
  standalone viewer with no issue.

  This is probably https://bugzilla.mozilla.org/show_bug.cgi?id=1003707

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/1315020/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to