Summary: | [GTK] Rename WebKitPrintOperation::done signal to ::finished in WebKit2 | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | gustavo | ||||
Priority: | P2 | Keywords: | Gtk | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Linux | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2012-02-17 05:41:37 PST
Created attachment 127578 [details]
Patch
It's not emitted anynmore when the print dialog has been cancelled. Since it's not possible to know whether the print dialog was cancelled or not, the enum WebKitPrintOperationResponse has been added, with Print and Cancel values, and it's used as return value of webkit_print_operation_run_dialog(). I've used an enum instead of a boolean just in case we add WEBKIT_PRINT_OPERATION_RESPONSE_PREVIEW in the future
Comment on attachment 127578 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=127578&action=review Looks good to me; also having an enum allows us to expand this API more easily, though I think it'll probably not need too much change in the future after it settles =) > Source/WebKit2/ChangeLog:18 > + (webkitPrintOperationRunDialogWin32): Dittoo. s/oo/o/ > Source/WebKit2/ChangeLog:20 > + (webkitPrintOperationRunDialogForFrame): Do ot emit finished if s/ot/not/ Committed r108069: <http://trac.webkit.org/changeset/108069> |