Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8225)

Unified Diff: chrome/browser/ui/webui/print_preview_ui.h

Issue 7550022: Print Preview: Fixing behavior of event listeners. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressing kmadhusu's comments Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/print_preview_ui.h
diff --git a/chrome/browser/ui/webui/print_preview_ui.h b/chrome/browser/ui/webui/print_preview_ui.h
index 192611bc16a2ede8d125b3838ad002fd82ade6d7..bae8d4e461772b4e6cc250ef46201a01ad33c37d 100644
--- a/chrome/browser/ui/webui/print_preview_ui.h
+++ b/chrome/browser/ui/webui/print_preview_ui.h
@@ -16,6 +16,7 @@
class PrintPreviewDataService;
class PrintPreviewHandler;
+struct PrintHostMsg_DidGetPreviewPageCount_Params;
class PrintPreviewUI : public ChromeWebUI {
public:
@@ -41,15 +42,16 @@ class PrintPreviewUI : public ChromeWebUI {
// OnPrintPreviewFailed().
void OnPrintPreviewRequest();
- // Notify the Web UI that the print preview will have |page_count| pages.
- // |is_modifiable| indicates if the preview can be rerendered with different
- // print settings.
- void OnDidGetPreviewPageCount(int document_cookie_,
- int page_count,
- bool is_modifiable);
+ // Notify the Web UI that the print preview will have |params.page_count|
+ // pages. |params.is_modifiable| indicates if the preview can be rerendered
+ // with different print settings. |params.preview_request_id| indicates wich
+ // request resulted in this response.
+ void OnDidGetPreviewPageCount(
+ const PrintHostMsg_DidGetPreviewPageCount_Params& params);
// Notify the Web UI that the 0-based page |page_number| has been rendered.
- void OnDidPreviewPage(int page_number);
+ // |preview_request_id| indicates wich request resulted in this response.
+ void OnDidPreviewPage(int page_number, int preview_request_id);
// Notify the Web UI renderer that preview data is available.
// |expected_pages_count| specifies the total number of pages.

Powered by Google App Engine
This is Rietveld 408576698