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

Unified Diff: chrome/service/cloud_print/print_system.h

Issue 566693002: Use file handles to interact with utility process. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Mon Sep 15 03:22:54 PDT 2014 Created 6 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: chrome/service/cloud_print/print_system.h
diff --git a/chrome/service/cloud_print/print_system.h b/chrome/service/cloud_print/print_system.h
index 2da5f31ecbed5b64fea932a2596e91eb8c467437..41cd258ab434a463fe695353d562fa5ab62038cc 100644
--- a/chrome/service/cloud_print/print_system.h
+++ b/chrome/service/cloud_print/print_system.h
@@ -75,11 +75,11 @@ class PrintSystem : public base::RefCountedThreadSafe<PrintSystem> {
// Callback interface for new printer notifications.
class Delegate {
public:
Lei Zhang 2014/09/15 21:49:53 nit: actually this is indented 1 space too many.
Vitaly Buka (NO REVIEWS) 2014/09/15 22:33:50 Done.
- virtual void OnPrinterAdded() = 0;
+ virtual void OnPrinterAdded() = 0;
// TODO(gene): Do we need OnPrinterDeleted notification here?
protected:
- virtual ~Delegate() {}
+ virtual ~Delegate() {}
};
virtual bool StartWatching(PrintServerWatcher::Delegate* delegate) = 0;
@@ -95,12 +95,12 @@ class PrintSystem : public base::RefCountedThreadSafe<PrintSystem> {
// Callback interface for printer updates notifications.
class Delegate {
public:
- virtual void OnPrinterDeleted() = 0;
- virtual void OnPrinterChanged() = 0;
- virtual void OnJobChanged() = 0;
+ virtual void OnPrinterDeleted() = 0;
+ virtual void OnPrinterChanged() = 0;
+ virtual void OnJobChanged() = 0;
protected:
- virtual ~Delegate() {}
+ virtual ~Delegate() {}
};
virtual bool StartWatching(PrinterWatcher::Delegate* delegate) = 0;

Powered by Google App Engine
This is Rietveld 408576698