summaryrefslogtreecommitdiffstats
path: root/chrome/browser/printing
diff options
context:
space:
mode:
authorerg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-10-14 16:53:41 +0000
committererg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2009-10-14 16:53:41 +0000
commit038d52e1cd7f5ec3eda5c18b4c4437c372ec9c04 (patch)
treece06e8884aebf474fcb02170d3af0334567fa8f4 /chrome/browser/printing
parentd76d4ebac33dd9df8c3aa361b7902e0a934fde4d (diff)
downloadchromium_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.cc2
-rw-r--r--chrome/browser/printing/print_job.cc3
-rw-r--r--chrome/browser/printing/print_job_unittest.cc2
-rw-r--r--chrome/browser/printing/printing_layout_uitest.cc2
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) {
}