Print Preview: Fixing behavior of event listeners.

chrome/browser/ui/webui/print_preview_ui.h

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

[kmadhusu, 2011/08/10 17:12:40]

nit: I think the struct definition has a detailed description about the member
variables. We don't want to repeat that here.

[dpapad, 2011/08/10 17:21:40]

Done. Removed the details from this comment.

+  // 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.