Printing: Keep track of WebContents lifetime for the native printing UI.

chrome/browser/printing/printer_query.h

 // Copyright (c) 2012 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_PRINTER_QUERY_H_
 #define CHROME_BROWSER_PRINTING_PRINTER_QUERY_H_
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/printing/print_job_worker_owner.h"
 #include "printing/print_job_constants.h"
-#include "ui/gfx/native_widget_types.h"
+
+class PrintingUIWebContentsObserver;
 
 namespace base {
 class DictionaryValue;
 class MessageLoop;
 }
 
 namespace printing {
 
-  class PrintDestinationInterface;
-  class PrintJobWorker;
+class PrintDestinationInterface;
+class PrintJobWorker;
 
 // Query the printer for settings.
 class PrinterQuery : public PrintJobWorkerOwner {
  public:
   // GetSettings() UI parameter.
   enum GetSettingsAskParam {
     DEFAULTS,
     ASK_USER,
   };
 
   PrinterQuery();
 
   // PrintJobWorkerOwner implementation.
   virtual void GetSettingsDone(const PrintSettings& new_settings,
                                PrintingContext::Result result) OVERRIDE;
   virtual PrintJobWorker* DetachWorker(PrintJobWorkerOwner* new_owner) OVERRIDE;
   virtual base::MessageLoop* message_loop() OVERRIDE;
   virtual const PrintSettings& settings() const OVERRIDE;
   virtual int cookie() const OVERRIDE;
 
   // Initializes the printing context. It is fine to call this function multiple
-  // times to reinitialize the settings. |parent_view| parameter's window will
-  // be the owner of the print setting dialog box. It is unused when
+  // times to reinitialize the settings. |web_contents_observer| can be queried
+  // to find the owner of the print setting dialog box. It is unused when
   // |ask_for_user_settings| is DEFAULTS.
-  void GetSettings(GetSettingsAskParam ask_user_for_settings,
-                   gfx::NativeView parent_view,
-                   int expected_page_count,
-                   bool has_selection,
-                   MarginType margin_type,
-                   const base::Closure& callback);
+  void GetSettings(
+      GetSettingsAskParam ask_user_for_settings,
+      scoped_ptr<PrintingUIWebContentsObserver> web_contents_observer,
+      int expected_page_count,
+      bool has_selection,
+      MarginType margin_type,
+      const base::Closure& callback);
 
   // Updates the current settings with |new_settings| dictionary values.
   void SetSettings(const base::DictionaryValue& new_settings,
                    const base::Closure& callback);
 
   // Set a destination for the worker.
   void SetWorkerDestination(PrintDestinationInterface* destination);
 
   // Stops the worker thread since the client is done with this object.
   void StopWorker();
@@ skipping 35 matching lines @@
 
   // Callback waiting to be run.
   base::Closure callback_;
 
   DISALLOW_COPY_AND_ASSIGN(PrinterQuery);
 };
 
 }  // namespace printing
 
 #endif  // CHROME_BROWSER_PRINTING_PRINTER_QUERY_H_