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

Unified Diff: chrome/browser/printing/print_dialog_cloud.h

Issue 10388214: Make cloud print dialog creation function not use browser list, instead obtain profile and parent w… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 7 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
« no previous file with comments | « chrome/browser/chrome_content_browser_client.cc ('k') | chrome/browser/printing/print_dialog_cloud.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/printing/print_dialog_cloud.h
===================================================================
--- chrome/browser/printing/print_dialog_cloud.h (revision 138829)
+++ chrome/browser/printing/print_dialog_cloud.h (working copy)
@@ -12,39 +12,55 @@
#include "base/callback.h"
#include "base/memory/ref_counted_memory.h"
#include "base/string16.h"
+#include "ui/gfx/native_widget_types.h"
class FilePath;
class CommandLine;
+namespace content {
+class BrowserContext;
+}
+
namespace print_dialog_cloud {
// Creates a print dialog to print a file on disk.
// Called on the FILE or UI thread. Even though this may start up a modal
// dialog, it will return immediately. The dialog is handled asynchronously.
-void CreatePrintDialogForFile(const FilePath& path_to_file,
+// If non-NULL, |modal_parent| specifies a window that the print dialog is modal
+// to.
+void CreatePrintDialogForFile(content::BrowserContext* browser_context,
+ gfx::NativeWindow modal_parent,
+ const FilePath& path_to_file,
const string16& print_job_title,
const string16& print_ticket,
const std::string& file_type,
- bool modal,
bool delete_on_close);
// Creates a print dialog to print data in RAM.
// Called on the FILE or UI thread. Even though this may start up a modal
// dialog, it will return immediately. The dialog is handled asynchronously.
-void CreatePrintDialogForBytes(scoped_refptr<base::RefCountedBytes> data,
+// If non-NULL, |modal_parent| specifies a window that the print dialog is modal
+// to.
+void CreatePrintDialogForBytes(content::BrowserContext* browser_context,
+ gfx::NativeWindow modal_parent,
+ scoped_refptr<base::RefCountedBytes> data,
const string16& print_job_title,
const string16& print_ticket,
- const std::string& file_type,
- bool modal);
+ const std::string& file_type);
// Parse switches from command_line and display the print dialog as appropriate.
+// Uses the default profile.
bool CreatePrintDialogFromCommandLine(const CommandLine& command_line);
// Creates a dialog for signing into cloud print.
// The dialog will call |callback| when complete.
// Called on the UI thread. Even though this starts up a modal
// dialog, it will return immediately. The dialog is handled asynchronously.
-void CreateCloudPrintSigninDialog(const base::Closure& callback);
+// If non-NULL, |modal_parent| specifies a window that the print dialog is modal
+// to.
+void CreateCloudPrintSigninDialog(content::BrowserContext* browser_context,
+ gfx::NativeWindow modal_parent,
+ const base::Closure& callback);
} // end namespace
« no previous file with comments | « chrome/browser/chrome_content_browser_client.cc ('k') | chrome/browser/printing/print_dialog_cloud.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698