diff options
author | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-03 00:56:01 +0000 |
---|---|---|
committer | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-03 00:56:01 +0000 |
commit | 0c6a76e4a58d81d125d18f9b576d0c44fcf3449f (patch) | |
tree | cec7974ed73a510cd6bb69976ba8e4563876748b /chrome/browser/printing/print_job_manager.h | |
parent | d5d62170b4464c8c41f51aa366fa67c7a4495f0b (diff) | |
download | chromium_src-0c6a76e4a58d81d125d18f9b576d0c44fcf3449f.zip chromium_src-0c6a76e4a58d81d125d18f9b576d0c44fcf3449f.tar.gz chromium_src-0c6a76e4a58d81d125d18f9b576d0c44fcf3449f.tar.bz2 |
Printing: Remove an unused variable, update some obsolete comments, and remove an unneeded prefs check.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8386028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108378 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/printing/print_job_manager.h')
-rw-r--r-- | chrome/browser/printing/print_job_manager.h | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/chrome/browser/printing/print_job_manager.h b/chrome/browser/printing/print_job_manager.h index 72548da..8d37537f 100644 --- a/chrome/browser/printing/print_job_manager.h +++ b/chrome/browser/printing/print_job_manager.h @@ -19,9 +19,7 @@ class PrefService; namespace printing { class JobEventDetails; -class PrintedDocument; class PrintJob; -class PrintedPage; class PrinterQuery; class PrintJobManager : public content::NotificationObserver { @@ -73,10 +71,14 @@ class PrintJobManager : public content::NotificationObserver { // Used to serialize access to queued_workers_. base::Lock lock_; - // Printing is enabled/disabled. This variable is checked at only one place, - // by RenderMessageFilter::OnGetDefaultPrintSettings. If its value is true + // Printing is enabled/disabled. For printing with the native print dialog, + // this variable is checked at only one place, by + // PrintingMessageFilter::OnGetDefaultPrintSettings. If its value is true // at that point, then the initiated print flow will complete itself, // even if the value of this variable changes afterwards. + // In the print preview workflow, this variable is checked in + // PrintingMessageFilter::OnUpdatePrintSettings, which gets called multiple + // times in the print preview workflow. BooleanPrefMember printing_enabled_; PrinterQueries queued_queries_; |