Move MockUpdateEngineClient::GetLastStatus() expectation to MockDBusThreadManager's ctor

chromeos/dbus/mock_dbus_thread_manager.cc

Index: chromeos/dbus/mock_dbus_thread_manager.cc
diff --git a/chromeos/dbus/mock_dbus_thread_manager.cc b/chromeos/dbus/mock_dbus_thread_manager.cc
index 99e1a35f4e994098aa57563f726edc7a77deab1b..10f3d86b21d7b5bf53a3e3a40b89bacb6815f406 100644
--- a/chromeos/dbus/mock_dbus_thread_manager.cc
+++ b/chromeos/dbus/mock_dbus_thread_manager.cc
@@ -273,6 +273,13 @@ MockDBusThreadManager::MockDBusThreadManager()
   EXPECT_CALL(*mock_shill_manager_client_.get(),
               RemovePropertyChangedObserver(_))
       .Times(AnyNumber());
+
+  // For CrOS browsertests, ChromeBrowserMainPartsChromeos::PostProfileInit()
+  // creates an AutomaticRebootManager which calls the following function.
+  // For unittests, this function won't get called.
+  EXPECT_CALL(*mock_update_engine_client_, GetLastStatus())
+      .Times(AnyNumber())

[Daniel Erat, 2013/04/10 20:06:00]

nit: is this .Times(AnyNumber()) necessary?  i think you may be able to leave it
out -- i'd expect WillRepeatedly() to imply it

[Yufeng Shen (Slow to review), 2013/04/10 20:29:21]

Done.

+      .WillRepeatedly(Return(MockUpdateEngineClient::Status()));
 }
 
 MockDBusThreadManager::~MockDBusThreadManager() {