Skip to content
Snippets Groups Projects
Commit ee42691e authored by jhawkins@chromium.org's avatar jhawkins@chromium.org
Browse files

Printing: Remove the debug-only |page_number_| from PrintingContext, which has

outlived its usefulness.

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/3417025

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60731 0039d316-1c4b-4281-b951-d872f2087c98
parent 08e27fba
No related branches found
No related tags found
No related merge requests found
......@@ -180,11 +180,6 @@ class PrintingContext {
// Complete print context settings.
PrintSettings settings_;
#ifndef NDEBUG
// Current page number in the print job.
int page_number_;
#endif
#if defined(OS_WIN)
// The dialog box for the time it is shown.
volatile HWND dialog_box_;
......
......@@ -13,9 +13,6 @@ namespace printing {
PrintingContext::PrintingContext()
:
#ifndef NDEBUG
page_number_(-1),
#endif
dialog_box_dismissed_(false),
in_print_job_(false),
abort_printing_(false) {
......@@ -66,9 +63,6 @@ PrintingContext::Result PrintingContext::InitWithSettings(
}
void PrintingContext::ResetSettings() {
#ifndef NDEBUG
page_number_ = -1;
#endif
dialog_box_dismissed_ = false;
abort_printing_ = false;
in_print_job_ = false;
......@@ -80,10 +74,6 @@ PrintingContext::Result PrintingContext::NewDocument(
NOTIMPLEMENTED();
#ifndef NDEBUG
page_number_ = 0;
#endif
return FAILED;
}
......@@ -94,10 +84,6 @@ PrintingContext::Result PrintingContext::NewPage() {
NOTIMPLEMENTED();
#ifndef NDEBUG
++page_number_;
#endif
return FAILED;
}
......
......@@ -15,9 +15,6 @@ namespace printing {
PrintingContext::PrintingContext()
: context_(NULL),
print_info_(nil),
#ifndef NDEBUG
page_number_(-1),
#endif
dialog_box_dismissed_(false),
in_print_job_(false),
abort_printing_(false) {
......@@ -114,9 +111,6 @@ void PrintingContext::ResetSettings() {
[print_info_ autorelease];
print_info_ = nil;
settings_.Clear();
#ifndef NDEBUG
page_number_ = -1;
#endif
dialog_box_dismissed_ = false;
abort_printing_ = false;
in_print_job_ = false;
......@@ -146,10 +140,6 @@ PrintingContext::Result PrintingContext::NewDocument(
if (status != noErr)
return OnError();
#ifndef NDEBUG
page_number_ = 0;
#endif
return OK;
}
......@@ -171,10 +161,6 @@ PrintingContext::Result PrintingContext::NewPage() {
if (status != noErr)
return OnError();
#ifndef NDEBUG
++page_number_;
#endif
return OK;
}
......
......@@ -122,9 +122,6 @@ class PrintingContext::CallbackHandler : public IPrintDialogCallback,
PrintingContext::PrintingContext()
: context_(NULL),
#ifndef NDEBUG
page_number_(-1),
#endif
dialog_box_(NULL),
dialog_box_dismissed_(false),
in_print_job_(false),
......@@ -240,10 +237,6 @@ void PrintingContext::ResetSettings() {
}
settings_.Clear();
in_print_job_ = false;
#ifndef NDEBUG
page_number_ = -1;
#endif
}
PrintingContext::Result PrintingContext::NewDocument(
......@@ -292,9 +285,6 @@ PrintingContext::Result PrintingContext::NewDocument(
if (StartDoc(context_, &di) <= 0)
return OnError();
#ifndef NDEBUG
page_number_ = 0;
#endif
return OK;
}
......@@ -309,10 +299,6 @@ PrintingContext::Result PrintingContext::NewPage() {
if (StartPage(context_) <= 0)
return OnError();
#ifndef NDEBUG
++page_number_;
#endif
return OK;
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment