Virtual destructors should have virtual keyword.

chrome/service/service_process.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_SERVICE_SERVICE_PROCESS_H_
 #define CHROME_SERVICE_SERVICE_PROCESS_H_
 #pragma once
 
 #include <string>
 
@@ skipping 14 matching lines @@
 class NetworkChangeNotifier;
 }
 
 class CommandLine;
 
 // The ServiceProcess does not inherit from ChildProcess because this
 // process can live independently of the browser process.
 class ServiceProcess : public CloudPrintProxy::Client {
  public:
   ServiceProcess();
-  ~ServiceProcess();
+  virtual ~ServiceProcess();
 
   // Initialize the ServiceProcess with the message loop that it should run on.
   // ServiceProcess takes ownership of |state|.
   bool Initialize(MessageLoopForUI* message_loop,
                   const CommandLine& command_line,
                   ServiceProcessState* state);
   bool Teardown();
   // TODO(sanjeevr): Change various parts of the code such as
   // net::ProxyService::CreateSystemProxyConfigService to take in
   // MessageLoopProxy* instead of MessageLoop*. When we have done that, we can
@@ skipping 83 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(ServiceProcess);
 };
 
 // Disable refcounting for runnable method because it is really not needed
 // when we post tasks on the main message loop.
 DISABLE_RUNNABLE_METHOD_REFCOUNT(ServiceProcess);
 
 extern ServiceProcess* g_service_process;
 
 #endif  // CHROME_SERVICE_SERVICE_PROCESS_H_