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

Unified Diff: printing/printing_context_win.cc

Issue 3417025: Printing: Remove the debug-only |page_number_| from PrintingContext, which has (Closed)
Patch Set: Created 10 years, 3 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 | « printing/printing_context_mac.mm ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: printing/printing_context_win.cc
diff --git a/printing/printing_context_win.cc b/printing/printing_context_win.cc
index 83ddfd4d44d94ee7fc02b9e5c08e8b4fe57614e7..5074b410c3b68b1e50c98cf96b34856f2f41913e 100644
--- a/printing/printing_context_win.cc
+++ b/printing/printing_context_win.cc
@@ -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;
}
« no previous file with comments | « printing/printing_context_mac.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698