Remove black magic and >100 lines. Unhook a lot of dead code.

chrome/browser/printing/print_view_manager.h

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_PRINTING_PRINT_VIEW_MANAGER_H_
 #define CHROME_BROWSER_PRINTING_PRINT_VIEW_MANAGER_H_
 
 #include "chrome/browser/printing/printed_pages_source.h"
 #include "chrome/common/notification_observer.h"
 #include "chrome/common/render_messages.h"
@@ skipping 15 matching lines @@
   PrintViewManager(WebContents& owner);
   virtual ~PrintViewManager();
 
   // Destroys the "Print..." dialog, makes sure the pages are finished rendering
   // and release the print job.
   void Destroy();
 
   // Cancels the print job.
   void Stop();
 
-  // Shows the "Print..." dialog if none is shown and if no rendering is
-  // pending. This is done asynchronously.
-  void ShowPrintDialog();
-
-  // Initiates a print job immediately. This is done asynchronously. Returns
-  // false if printing is impossible at the moment.
-  bool PrintNow();
-
   // Terminates or cancels the print job if one was pending, depending on the
   // current state. Returns false if the renderer was not valuable.
   bool OnRendererGone(RenderViewHost* render_view_host);
 
   // Received a notification from the renderer that the number of printed page
   // has just been calculated..
   void DidGetPrintedPagesCount(int cookie, int number_pages);
 
   // Received a notification from the renderer that a printed page page is
   // finished renderering.
   void DidPrintPage(const ViewHostMsg_DidPrintPage_Params& params);
 
   // PrintedPagesSource implementation.
-  virtual void RenderOnePrintedPage(PrintedDocument* document, int page_number);
   virtual std::wstring RenderSourceName();
   virtual GURL RenderSourceUrl();
 
   // NotificationObserver implementation.
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
                        const NotificationDetails& details);
 
  private:
   // Processes a NOTIFY_PRINT_JOB_EVENT notification.
   void OnNotifyPrintJobEvent(const JobEventDetails& event_details);
 
-  // Processes a xxx_INIT_xxx type of NOTIFY_PRINT_JOB_EVENT notification.
-  void OnNotifyPrintJobInitEvent(const JobEventDetails& event_details);
-
   // Requests the RenderView to render all the missing pages for the print job.
   // Noop if no print job is pending. Returns true if at least one page has been
   // requested to the renderer.
   bool RenderAllMissingPagesNow();
 
   // Quits the current message loop if these conditions hold true: a document is
   // loaded and is complete and waiting_for_pages_to_be_rendered_ is true. This
   // function is called in DidPrintPage() or on ALL_PAGES_REQUESTED
   // notification. The inner message loop is created was created by
   // RenderAllMissingPagesNow().
@@ skipping 44 matching lines @@
   // Waiting for print_job_ initialization to be completed to start printing.
   // Specifically the DEFAULT_INIT_DONE notification. Set when PrintNow() is
   // called.
   bool waiting_to_print_;
 
   // Running an inner message loop inside RenderAllMissingPagesNow(). This means
   // we are _blocking_ until all the necessary pages have been rendered or the
   // print settings are being loaded.
   bool inside_inner_message_loop_;
 
-  // The object is waiting for some information to call print_job_->Init(true).
-  // It is either a DEFAULT_INIT_DONE notification or the
-  // DidGetPrintedPagesCount() callback.
-  // Showing the Print... dialog box is a multi-step operation:
-  // - print_job_->Init(false) to get the default settings. Set
-  //   waiting_to_show_print_dialog_ = true.
-  // - on DEFAULT_INIT_DONE, gathers new settings.
-  // - did settings() or document() change since the last intialization?
-  //   - Call SwitchCssToPrintMediaType()
-  //     - On DidGetPrintedPagesCount() call, if
-  //       waiting_to_show_print_dialog_ = true
-  //       - calls print_job_->Init(true).
-  //       - waiting_to_show_print_dialog_ = false.
-  //       - DONE.
-  // - else (It may happens when redisplaying the dialog box with settings that
-  //         haven't changed)
-  //   - if waiting_to_show_print_dialog_ = true && page_count is initialized.
-  //     - calls print_job_->Init(true).
-  //     - waiting_to_show_print_dialog_ = false.
-  bool waiting_to_show_print_dialog_;
-
   // PrintViewManager is created as an extension of WebContent specialized for
   // printing-related behavior. Still, access to the renderer is needed so a
   // back reference is kept the the "parent object".
   WebContents& owner_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintViewManager);
 };
 
 }  // namespace printing
 
 #endif  // CHROME_BROWSER_PRINTING_PRINT_VIEW_MANAGER_H_