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

Side by Side Diff: chrome/browser/printing/print_system_task_proxy.cc

Issue 15987009: Update chrome/ to use WeakPtr<T>::get() instead of implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/printing/print_system_task_proxy.h" 5 #include "chrome/browser/printing/print_system_task_proxy.h"
6 6
7 #include <ctype.h> 7 #include <ctype.h>
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 BrowserThread::PostTask( 56 BrowserThread::PostTask(
57 BrowserThread::UI, FROM_HERE, 57 BrowserThread::UI, FROM_HERE,
58 base::Bind(&PrintSystemTaskProxy::SendDefaultPrinter, this, 58 base::Bind(&PrintSystemTaskProxy::SendDefaultPrinter, this,
59 print_backend_->GetDefaultPrinterName(), 59 print_backend_->GetDefaultPrinterName(),
60 std::string())); 60 std::string()));
61 } 61 }
62 62
63 void PrintSystemTaskProxy::SendDefaultPrinter( 63 void PrintSystemTaskProxy::SendDefaultPrinter(
64 const std::string& default_printer, 64 const std::string& default_printer,
65 const std::string& cloud_print_data) { 65 const std::string& cloud_print_data) {
66 if (handler_) 66 if (handler_.get())
67 handler_->SendInitialSettings(default_printer, cloud_print_data); 67 handler_->SendInitialSettings(default_printer, cloud_print_data);
68 } 68 }
69 69
70 void PrintSystemTaskProxy::EnumeratePrinters() { 70 void PrintSystemTaskProxy::EnumeratePrinters() {
71 VLOG(1) << "Enumerate printers start"; 71 VLOG(1) << "Enumerate printers start";
72 ListValue* printers = new ListValue; 72 ListValue* printers = new ListValue;
73 printing::PrinterList printer_list; 73 printing::PrinterList printer_list;
74 print_backend_->EnumeratePrinters(&printer_list); 74 print_backend_->EnumeratePrinters(&printer_list);
75 75
76 if (!has_logged_printers_count_) { 76 if (!has_logged_printers_count_) {
(...skipping 20 matching lines...) Expand all
97 printers->Append(printer_info); 97 printers->Append(printer_info);
98 } 98 }
99 VLOG(1) << "Enumerate printers finished, found " << i << " printers"; 99 VLOG(1) << "Enumerate printers finished, found " << i << " printers";
100 100
101 BrowserThread::PostTask( 101 BrowserThread::PostTask(
102 BrowserThread::UI, FROM_HERE, 102 BrowserThread::UI, FROM_HERE,
103 base::Bind(&PrintSystemTaskProxy::SetupPrinterList, this, printers)); 103 base::Bind(&PrintSystemTaskProxy::SetupPrinterList, this, printers));
104 } 104 }
105 105
106 void PrintSystemTaskProxy::SetupPrinterList(ListValue* printers) { 106 void PrintSystemTaskProxy::SetupPrinterList(ListValue* printers) {
107 if (handler_) 107 if (handler_.get())
108 handler_->SetupPrinterList(*printers); 108 handler_->SetupPrinterList(*printers);
109 delete printers; 109 delete printers;
110 } 110 }
111 111
112 void PrintSystemTaskProxy::GetPrinterCapabilities( 112 void PrintSystemTaskProxy::GetPrinterCapabilities(
113 const std::string& printer_name) { 113 const std::string& printer_name) {
114 VLOG(1) << "Get printer capabilities start for " << printer_name; 114 VLOG(1) << "Get printer capabilities start for " << printer_name;
115 child_process_logging::ScopedPrinterInfoSetter prn_info( 115 child_process_logging::ScopedPrinterInfoSetter prn_info(
116 print_backend_->GetPrinterDriverInfo(printer_name)); 116 print_backend_->GetPrinterDriverInfo(printer_name));
117 117
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 } 155 }
156 156
157 BrowserThread::PostTask( 157 BrowserThread::PostTask(
158 BrowserThread::UI, FROM_HERE, 158 BrowserThread::UI, FROM_HERE,
159 base::Bind(&PrintSystemTaskProxy::SendPrinterCapabilities, this, 159 base::Bind(&PrintSystemTaskProxy::SendPrinterCapabilities, this,
160 settings_info.DeepCopy())); 160 settings_info.DeepCopy()));
161 } 161 }
162 162
163 void PrintSystemTaskProxy::SendPrinterCapabilities( 163 void PrintSystemTaskProxy::SendPrinterCapabilities(
164 DictionaryValue* settings_info) { 164 DictionaryValue* settings_info) {
165 if (handler_) 165 if (handler_.get())
166 handler_->SendPrinterCapabilities(*settings_info); 166 handler_->SendPrinterCapabilities(*settings_info);
167 delete settings_info; 167 delete settings_info;
168 } 168 }
169 169
170 void PrintSystemTaskProxy::SendFailedToGetPrinterCapabilities( 170 void PrintSystemTaskProxy::SendFailedToGetPrinterCapabilities(
171 const std::string& printer_name) { 171 const std::string& printer_name) {
172 if (handler_) 172 if (handler_.get())
173 handler_->SendFailedToGetPrinterCapabilities(printer_name); 173 handler_->SendFailedToGetPrinterCapabilities(printer_name);
174 } 174 }
OLDNEW
« no previous file with comments | « chrome/browser/printing/print_dialog_cloud_unittest.cc ('k') | chrome/browser/search_engines/search_provider_install_data.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698