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

Unified Diff: trunk/src/chrome/service/cloud_print/print_system_win.cc

Issue 23503070: Revert 223610 "Set the printer info in crash reports using the c..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 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
Index: trunk/src/chrome/service/cloud_print/print_system_win.cc
===================================================================
--- trunk/src/chrome/service/cloud_print/print_system_win.cc (revision 223612)
+++ trunk/src/chrome/service/cloud_print/print_system_win.cc (working copy)
@@ -18,7 +18,7 @@
#include "base/win/scoped_com_initializer.h"
#include "base/win/scoped_comptr.h"
#include "base/win/scoped_hdc.h"
-#include "chrome/common/crash_keys.h"
+#include "chrome/common/child_process_logging.h"
#include "chrome/service/service_process.h"
#include "chrome/service/service_utility_process_host.h"
#include "grit/generated_resources.h"
@@ -170,7 +170,7 @@
scoped_refptr<printing::PrintBackend> print_backend(
printing::PrintBackend::CreateInstance(NULL));
printer_info_ = print_backend->GetPrinterDriverInfo(printer_name);
- crash_keys::ScopedPrinterInfo crash_key(printer_info_);
+ child_process_logging::ScopedPrinterInfoSetter prn_info(printer_info_);
delegate_ = delegate;
// An empty printer name means watch the current server, we need to pass
@@ -203,7 +203,7 @@
// base::ObjectWatcher::Delegate method
virtual void OnObjectSignaled(HANDLE object) {
- crash_keys::ScopedPrinterInfo crash_key(printer_info_);
+ child_process_logging::ScopedPrinterInfoSetter prn_info(printer_info_);
DWORD change = 0;
FindNextPrinterChangeNotification(object, &change, NULL, NULL);
@@ -370,7 +370,7 @@
// TODO(gene): add tags handling.
scoped_refptr<printing::PrintBackend> print_backend(
printing::PrintBackend::CreateInstance(NULL));
- crash_keys::ScopedPrinterInfo crash_key(
+ child_process_logging::ScopedPrinterInfoSetter prn_info(
print_backend->GetPrinterDriverInfo(printer_name));
return core_->Spool(print_ticket, print_data_file_path,
print_data_mime_type, printer_name, job_title,
@@ -403,7 +403,7 @@
JobSpooler::Delegate* delegate) {
scoped_refptr<printing::PrintBackend> print_backend(
printing::PrintBackend::CreateInstance(NULL));
- crash_keys::ScopedPrinterInfo crash_key(
+ child_process_logging::ScopedPrinterInfoSetter prn_info(
print_backend->GetPrinterDriverInfo(printer_name));
if (delegate_) {
// We are already in the process of printing.
@@ -792,7 +792,7 @@
bool PrintSystemWin::ValidatePrintTicket(
const std::string& printer_name,
const std::string& print_ticket_data) {
- crash_keys::ScopedPrinterInfo crash_key(
+ child_process_logging::ScopedPrinterInfoSetter prn_info(
print_backend_->GetPrinterDriverInfo(printer_name));
printing::ScopedXPSInitializer xps_initializer;
if (!xps_initializer.initialized()) {
@@ -833,7 +833,7 @@
bool PrintSystemWin::GetJobDetails(const std::string& printer_name,
PlatformJobId job_id,
PrintJobDetails *job_details) {
- crash_keys::ScopedPrinterInfo crash_key(
+ child_process_logging::ScopedPrinterInfoSetter prn_info(
print_backend_->GetPrinterDriverInfo(printer_name));
DCHECK(job_details);
printing::ScopedPrinterHandle printer_handle;
« no previous file with comments | « trunk/src/chrome/service/cloud_print/print_system_cups.cc ('k') | trunk/src/chrome/utility/chrome_content_utility_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698