Fixed loader issue in unit tests caused by some test not using that mock and ...

chrome/browser/chromeos/cros/cros_library.cc

 // Copyright (c) 2009 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/cros/cros_library.h"
 
 #include "chrome/browser/chromeos/cros/cros_library_loader.h"
 #include "chrome/browser/chromeos/cros/cryptohome_library.h"
 #include "chrome/browser/chromeos/cros/language_library.h"
 #include "chrome/browser/chromeos/cros/login_library.h"
@@ skipping 96 matching lines @@
   return loaded_;
 }
 
 CrosLibrary::TestApi* CrosLibrary::GetTestApi() {
   if (!test_api_)
     test_api_ = new TestApi(this);
   return test_api_;
 }
 
 void CrosLibrary::TestApi::SetLibraryLoader(LibraryLoader* loader) {
+  if (library_->library_loader_ == loader)
+    return;
   if (library_->library_loader_)
     delete library_->library_loader_;
   library_->library_loader_ = loader;
+  // Reset load flags when loader changes. Otherwise some tests are really not
+  // going to be happy.
+  library_->loaded_ = false;
+  library_->load_error_ = false;
 }
 
 void CrosLibrary::TestApi::SetCryptohomeLibrary(CryptohomeLibrary* library) {
   if (library_->crypto_lib_)
     delete library_->crypto_lib_;
   library_->crypto_lib_ = library;
 }
 
 void CrosLibrary::TestApi::SetLanguageLibrary(LanguageLibrary* library) {
   if (library_->language_lib_)
@@ skipping 26 matching lines @@
 }
 
 void CrosLibrary::TestApi::SetSynapticsLibrary(SynapticsLibrary* library) {
   if (library_->synaptics_lib_)
     delete library_->synaptics_lib_;
   library_->synaptics_lib_ = library;
 }
 
 }   // end namespace.