[Webkit-unassigned] [Bug 22898] [GTK] Patch to allow batch printing to file

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=22898


colin at colino.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #26101|review?                     |
               Flag|                            |
  Attachment #26101|0                           |1
        is obsolete|                            |
  Attachment #26119|                            |review?
               Flag|                            |




------- Comment #6 from colin at colino.net  2008-12-18 01:02 PDT -------
Created an attachment (id=26119)
 --> (https://bugs.webkit.org/attachment.cgi?id=26119&action=view)
Patch that completes and makes GtkPrint API public

Hello,

Here's a rather bigger patch that aims to complete the GtkPrint API and make it
public.

What it does:
- WebKit/gtk/webkit/webkitwebframe.cpp: Factorizes code between print and
print_to_file, as little of it changes in both cases; Removes the error dialog,
leaving error handling to the caller; Adds the ability to specify page setup
and print settings in the Print case too, which lets the calling app have a
"Page setup" menu, and let it get back updated print settings in order to
remember them. This adds a few parameters to the functions, but helps
applications to have a full implementation; and add my name to the Copyright
boiler-plate.
- WebKit/gtk/webkit/webkitwebframe.h: Make the API public; the only annoyance
is that the API will be different whether compiled against GTK 2.10+ or not, as
the parameters type didn't exist before, it's not possible anymore to have
filler-functions printing a g_warning().
- WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp: Update code to use the new
API, with the minimal changes needed - just error handling the same way as it
was before, only at the caller level.

I tried to think of possible shortcomings to this API, because we wouldn't want
to modify a public API, but I think it's complete. _print_to_file() shows no
dialog, so there's no possibility to have updated settings after. GtkPageSetup
can't be modified from inside a print dialog, so there are no modifications to
get.


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.