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

Unified Diff: chrome/browser/printing/print_job_manager.h

Issue 149148: Remove unused notification in print code to simplify before refactoring.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 6 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 | « chrome/browser/printing/print_job.cc ('k') | chrome/browser/printing/print_job_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/printing/print_job_manager.h
===================================================================
--- chrome/browser/printing/print_job_manager.h (revision 19830)
+++ chrome/browser/printing/print_job_manager.h (working copy)
@@ -42,18 +42,6 @@
const NotificationSource& source,
const NotificationDetails& details);
- // Sets a path where to dump EMF data files. This enables debug behavior where
- // every rendered pages are dumped as-is. By default the path is empty, which
- // disables the dumping.
- // TODO(maruel): Remove me once printing is awesome.
- void set_debug_dump_path(const std::wstring& debug_dump_path) {
- debug_dump_path_ = debug_dump_path;
- }
-
- const std::wstring& debug_dump_path() const {
- return debug_dump_path_;
- }
-
private:
typedef std::vector<scoped_refptr<PrintJob> > PrintJobs;
typedef std::vector<scoped_refptr<PrinterQuery> > PrinterQueries;
@@ -62,12 +50,6 @@
void OnPrintJobEvent(PrintJob* print_job,
const JobEventDetails& event_details);
- // Processes a NOTIFY_PRINTED_DOCUMENT_UPDATED notification. When
- // debug_dump_path_ is not empty, it is processed to detect newly rendered
- // pages and to dump their EMF buffer.
- void OnPrintedDocumentUpdated(const PrintedDocument& document,
- const PrintedPage& page);
-
NotificationRegistrar registrar_;
// Used to serialize access to queued_workers_.
@@ -78,11 +60,6 @@
// Current print jobs that are active.
PrintJobs current_jobs_;
- // Path where debug dump of EMF buffer are saved. Empty by default. When
- // empty, EMF dumping is disabled.
- // TODO(maruel): Remove me once printing is awesome.
- std::wstring debug_dump_path_;
-
DISALLOW_EVIL_CONSTRUCTORS(PrintJobManager);
};
« no previous file with comments | « chrome/browser/printing/print_job.cc ('k') | chrome/browser/printing/print_job_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698