RefCounted types should not have public destructors, chromeos edition

chrome/browser/chromeos/login/login_utils.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/chromeos/login/login_utils.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "ash/ash_switches.h"
@@ skipping 179 matching lines @@
       // would be executed after committing pending write on file thread.
       BrowserThread::PostTask(
           BrowserThread::FILE, FROM_HERE,
           base::Bind(&JobRestartRequest::RestartJob, this));
     } else {
       RestartJob();
     }
   }
 
  private:
+  friend class base::RefCountedThreadSafe<JobRestartRequest>;
+
+  ~JobRestartRequest() {}
+
   void RestartJob() {
     if (BrowserThread::CurrentlyOn(BrowserThread::UI)) {
       DBusThreadManager::Get()->GetSessionManagerClient()->RestartJob(
           pid_, command_line_);
     } else {
       // This function can be called on FILE thread. See PostTask in the
       // constructor.
       BrowserThread::PostTask(
           BrowserThread::UI, FROM_HERE,
           base::Bind(&JobRestartRequest::RestartJob, this));
@@ skipping 904 matching lines @@
 bool LoginUtils::IsWhitelisted(const std::string& username) {
   CrosSettings* cros_settings = CrosSettings::Get();
   bool allow_new_user = false;
   cros_settings->GetBoolean(kAccountsPrefAllowNewUser, &allow_new_user);
   if (allow_new_user)
     return true;
   return cros_settings->FindEmailInList(kAccountsPrefUsers, username);
 }
 
 }  // namespace chromeos