cros: Slightly more mock out-of-lining.

chrome/browser/chromeos/login/owner_manager_unittest.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_BROWSER_CHROMEOS_LOGIN_OWNER_MANAGER_UNITTEST_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_OWNER_MANAGER_UNITTEST_H_
 
 #include "chrome/browser/chromeos/login/owner_manager.h"
 
 #include <string>
@@ skipping 21 matching lines @@
         this,
         NotificationType::OWNER_KEY_FETCH_ATTEMPT_FAILED,
         NotificationService::AllSources());
     registrar_.Add(
         this,
         NotificationType::OWNER_KEY_FETCH_ATTEMPT_SUCCEEDED,
         NotificationService::AllSources());
   }
 
   virtual ~MockKeyLoadObserver() {
-    EXPECT_TRUE(observed_);
+    DCHECK(observed_);
   }
 
   // NotificationObserver implementation.
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
-                       const NotificationDetails& details) {
-    LOG(INFO) << "Observed key fetch event";
-    if (type == NotificationType::OWNER_KEY_FETCH_ATTEMPT_SUCCEEDED) {
-      EXPECT_TRUE(success_expected_);
-      observed_ = true;
-      if (quit_on_observe_)
-        MessageLoop::current()->Quit();
-    } else if (type == NotificationType::OWNER_KEY_FETCH_ATTEMPT_FAILED) {
-      EXPECT_FALSE(success_expected_);
-      observed_ = true;
-      if (quit_on_observe_)
-        MessageLoop::current()->Quit();
-    }
-  }
+                       const NotificationDetails& details);
 
   void ExpectKeyFetchSuccess(bool should_succeed) {
     success_expected_ = should_succeed;
   }
 
   void SetQuitOnKeyFetch(bool should_quit) { quit_on_observe_ = should_quit; }
 
  private:
   NotificationRegistrar registrar_;
   bool success_expected_;
   bool quit_on_observe_;
   bool observed_;
   DISALLOW_COPY_AND_ASSIGN(MockKeyLoadObserver);
 };
 
 class MockKeyUser : public OwnerManager::Delegate {
  public:
   explicit MockKeyUser(const OwnerManager::KeyOpCode expected)
       : expected_(expected),
         quit_on_callback_(true) {
   }
   MockKeyUser(const OwnerManager::KeyOpCode expected, bool quit_on_callback)
       : expected_(expected),
         quit_on_callback_(quit_on_callback) {
   }
 
   virtual ~MockKeyUser() {}
 
-  void OnKeyOpComplete(const OwnerManager::KeyOpCode return_code,
-                       const std::vector<uint8>& payload) {
-    EXPECT_EQ(expected_, return_code);
-    if (quit_on_callback_)
-      MessageLoop::current()->Quit();
-  }
+  virtual void OnKeyOpComplete(const OwnerManager::KeyOpCode return_code,
+                       const std::vector<uint8>& payload);
 
   const OwnerManager::KeyOpCode expected_;
   const bool quit_on_callback_;
  private:
   DISALLOW_COPY_AND_ASSIGN(MockKeyUser);
 };
 
 class MockKeyUpdateUser : public OwnerManager::KeyUpdateDelegate {
  public:
   MockKeyUpdateUser() {}
@@ skipping 29 matching lines @@
   const OwnerManager::KeyOpCode expected_code_;
   const std::vector<uint8> expected_sig_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockSigner);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_OWNER_MANAGER_UNITTEST_H_