diff options
author | erg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-14 16:53:41 +0000 |
---|---|---|
committer | erg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-14 16:53:41 +0000 |
commit | 038d52e1cd7f5ec3eda5c18b4c4437c372ec9c04 (patch) | |
tree | ce06e8884aebf474fcb02170d3af0334567fa8f4 /chrome/browser/printing | |
parent | d76d4ebac33dd9df8c3aa361b7902e0a934fde4d (diff) | |
download | chromium_src-038d52e1cd7f5ec3eda5c18b4c4437c372ec9c04.zip chromium_src-038d52e1cd7f5ec3eda5c18b4c4437c372ec9c04.tar.gz chromium_src-038d52e1cd7f5ec3eda5c18b4c4437c372ec9c04.tar.bz2 |
More style nits.
(Working on cleaning chrome/ so hopefully we can lint it by default.)
Review URL: http://codereview.chromium.org/274040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28971 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/printing')
-rw-r--r-- | chrome/browser/printing/print_dialog_gtk.cc | 2 | ||||
-rw-r--r-- | chrome/browser/printing/print_job.cc | 3 | ||||
-rw-r--r-- | chrome/browser/printing/print_job_unittest.cc | 2 | ||||
-rw-r--r-- | chrome/browser/printing/printing_layout_uitest.cc | 2 |
4 files changed, 4 insertions, 5 deletions
diff --git a/chrome/browser/printing/print_dialog_gtk.cc b/chrome/browser/printing/print_dialog_gtk.cc index 14cc54e..7536a59 100644 --- a/chrome/browser/printing/print_dialog_gtk.cc +++ b/chrome/browser/printing/print_dialog_gtk.cc @@ -22,7 +22,7 @@ namespace { // to print to printers that don't support PDF. class PdfUnsupportedInfoBarDelegate : public LinkInfoBarDelegate { public: - PdfUnsupportedInfoBarDelegate(Browser* browser) + explicit PdfUnsupportedInfoBarDelegate(Browser* browser) : LinkInfoBarDelegate(NULL), browser_(browser) { } diff --git a/chrome/browser/printing/print_job.cc b/chrome/browser/printing/print_job.cc index 8a1a5e2..245c01a 100644 --- a/chrome/browser/printing/print_job.cc +++ b/chrome/browser/printing/print_job.cc @@ -317,8 +317,7 @@ void PrintJob::ControlledWorkerShutdown() { DispatchMessage(&msg); } // Continue looping. - } - else if (result == WAIT_OBJECT_0) { + } else if (result == WAIT_OBJECT_0) { // The thread quit. break; } else { diff --git a/chrome/browser/printing/print_job_unittest.cc b/chrome/browser/printing/print_job_unittest.cc index 3ef795c..8c79e6a 100644 --- a/chrome/browser/printing/print_job_unittest.cc +++ b/chrome/browser/printing/print_job_unittest.cc @@ -63,7 +63,7 @@ class TestOwner : public printing::PrintJobWorkerOwner { class TestPrintJob : public printing::PrintJob { public: - TestPrintJob(volatile bool* check) : check_(check) { + explicit TestPrintJob(volatile bool* check) : check_(check) { } ~TestPrintJob() { *check_ = true; diff --git a/chrome/browser/printing/printing_layout_uitest.cc b/chrome/browser/printing/printing_layout_uitest.cc index 80093fe..fb0b6c0 100644 --- a/chrome/browser/printing/printing_layout_uitest.cc +++ b/chrome/browser/printing/printing_layout_uitest.cc @@ -262,7 +262,7 @@ bool CloseDialogWindow(HWND dialog_window) { // default button. class DismissTheWindow : public base::DelegateSimpleThread::Delegate { public: - DismissTheWindow(DWORD owner_process) + explicit DismissTheWindow(DWORD owner_process) : owner_process_(owner_process) { } |