Make cloud print dialog creation function not use browser list, instead obtain profile and parent w…

chrome/browser/ui/webui/print_preview/print_preview_handler.cc

 // 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.
 
 #include "chrome/browser/ui/webui/print_preview/print_preview_handler.h"
 
 #include <ctype.h>
 
 #include <string>
 #include <vector>
@@ skipping 29 matching lines @@
 #include "chrome/browser/ui/webui/print_preview/print_preview_ui.h"
 #include "chrome/browser/ui/webui/print_preview/sticky_settings.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/print_messages.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
+#include "content/public/browser/web_contents_view.h"
 #include "content/public/browser/web_contents_delegate.h"
 #include "content/public/browser/web_ui.h"
 #include "printing/backend/print_backend.h"
 #include "printing/metafile.h"
 #include "printing/metafile_impl.h"
 #include "printing/page_range.h"
 #include "printing/page_size_margins.h"
 #include "printing/print_settings.h"
 #include "unicode/ulocdata.h"
 
@@ skipping 480 matching lines @@
     const base::WeakPtr<PrintPreviewHandler>& handler) {
   if (handler.get()) {
     PrintPreviewUI* print_preview_ui =
         static_cast<PrintPreviewUI*>(handler->web_ui()->GetController());
     if (print_preview_ui)
       print_preview_ui->OnReloadPrintersList();
   }
 }
 
 void PrintPreviewHandler::HandleSignin(const ListValue* /*args*/) {
+  gfx::NativeWindow modal_parent =
+      web_ui()->GetWebContents()->GetView()->GetTopLevelNativeWindow();
   print_dialog_cloud::CreateCloudPrintSigninDialog(
+      web_ui()->GetWebContents()->GetBrowserContext(),
+      modal_parent,
       base::Bind(&PrintPreviewHandler::OnSigninComplete, AsWeakPtr()));
 }
 
 void PrintPreviewHandler::HandlePrintWithCloudPrint() {
   // Record the number of times the user asks to print via cloud print
   // instead of the print preview dialog.
   ReportStats();
 
   PrintPreviewUI* print_preview_ui = static_cast<PrintPreviewUI*>(
       web_ui()->GetController());
   scoped_refptr<base::RefCountedBytes> data;
   print_preview_ui->GetPrintPreviewDataForIndex(
       printing::COMPLETE_PREVIEW_DOCUMENT_INDEX, &data);
   if (!data.get()) {
     NOTREACHED();
     return;
   }
   DCHECK_GT(data->size(), 0U);
-  print_dialog_cloud::CreatePrintDialogForBytes(data,
+
+  gfx::NativeWindow modal_parent =
+      web_ui()->GetWebContents()->GetView()->GetTopLevelNativeWindow();
+  print_dialog_cloud::CreatePrintDialogForBytes(
+      web_ui()->GetWebContents()->GetBrowserContext(),
+      modal_parent,
+      data,
       string16(print_preview_ui->initiator_tab_title()),
       string16(),
-      std::string("application/pdf"),
-      true);
+      std::string("application/pdf"));
 
   // Once the cloud print dialog comes up we're no longer in a background
   // printing situation.  Close the print preview.
   // TODO(abodenha@chromium.org) The flow should be changed as described in
   // http://code.google.com/p/chromium/issues/detail?id=44093
   ActivateInitiatorTabAndClosePreviewTab();
 }
 
 void PrintPreviewHandler::HandleManageCloudPrint(const ListValue* /*args*/) {
   ++manage_cloud_printers_dialog_request_count_;
@@ skipping 321 matching lines @@
     return;
 
   // We no longer require the initiator tab details. Remove those details
   // associated with the preview tab to allow the initiator tab to create
   // another preview tab.
   printing::PrintPreviewTabController* tab_controller =
       printing::PrintPreviewTabController::GetInstance();
   if (tab_controller)
     tab_controller->EraseInitiatorTabInfo(preview_tab_wrapper());
 }