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

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 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 368a3fa5798c538bb0658fc8d56d044dc436f0a5..8492e8534f8c8039c44181cb0473b67f98e6cadc 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:
@@ -40,15 +41,13 @@ class PrintPreviewUI : public ChromeWebUI {
// a matching call to OnPreviewDataIsAvailable() or OnPrintPreviewFailed().
void OnPrintPreviewRequest();
- // Notifies 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);
+ // Notifies the Web UI about the page count of the request preview.
+ void OnDidGetPreviewPageCount(
+ const PrintHostMsg_DidGetPreviewPageCount_Params& params);
// Notifies 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);
// Notifies the Web UI renderer that preview data is available.
// |expected_pages_count| specifies the total number of pages. |job_title| is
« no previous file with comments | « chrome/browser/resources/print_preview/print_preview_utils.js ('k') | chrome/browser/ui/webui/print_preview_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698