Incorporate comments in Dbus code and add Eol icon

chromeos/dbus/fake_update_engine_client.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 "chromeos/dbus/fake_update_engine_client.h"
 
+#include "base/bind.h"
+#include "base/threading/thread_task_runner_handle.h"
+
 namespace chromeos {
 
 FakeUpdateEngineClient::FakeUpdateEngineClient()
     : update_check_result_(UpdateEngineClient::UPDATE_RESULT_SUCCESS),
       can_rollback_stub_result_(false),
       reboot_after_update_call_count_(0),
       request_update_check_call_count_(0),
       rollback_call_count_(0),
-      can_rollback_call_count_(0) {
-}
+      can_rollback_call_count_(0),
+      weak_ptr_factory_(this) {}
 
 FakeUpdateEngineClient::~FakeUpdateEngineClient() {
 }
 
 void FakeUpdateEngineClient::Init(dbus::Bus* bus) {
 }
 
 void FakeUpdateEngineClient::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }
@@ skipping 39 matching lines @@
     const UpdateEngineClient::Status& status) {
   FOR_EACH_OBSERVER(Observer, observers_, UpdateStatusChanged(status));
 }
 
 void FakeUpdateEngineClient::SetChannel(const std::string& target_channel,
                                         bool is_powerwash_allowed) {
 }
 
 void FakeUpdateEngineClient::GetChannel(bool get_current_channel,
                                         const GetChannelCallback& callback) {
+  base::ThreadTaskRunnerHandle::Get()->PostTask(
+      FROM_HERE, base::Bind(&FakeUpdateEngineClient::OnGetChannel,
+                            weak_ptr_factory_.GetWeakPtr(), callback));
 }
 
 void FakeUpdateEngineClient::GetEolStatus(
-    const GetEolStatusCallback& callback) {}
+    const GetEolStatusCallback& callback) {
+  base::ThreadTaskRunnerHandle::Get()->PostTask(
+      FROM_HERE, base::Bind(&FakeUpdateEngineClient::OnGetEolStatus,
+                            weak_ptr_factory_.GetWeakPtr(), callback));
+}
 
 void FakeUpdateEngineClient::set_default_status(
     const UpdateEngineClient::Status& status) {
   default_status_ = status;
 }
 
 void FakeUpdateEngineClient::set_update_check_result(
     const UpdateEngineClient::UpdateCheckResult& result) {
   update_check_result_ = result;
 }
 
+void FakeUpdateEngineClient::OnGetChannel(const GetChannelCallback& callback) {

[hashimoto, 2016/06/23 05:12:48]

You don't need these methods and weak_ptr_factory_.

You should just base::Bind(callback, std::string()) and base::Bind(callback,
update_engine::EndOfLifeStatus::kSupported) instead.

+  callback.Run("");
+}
+
+void FakeUpdateEngineClient::OnGetEolStatus(
+    const GetEolStatusCallback& callback) {
+  callback.Run(update_engine::EndOfLifeStatus::kSupported);
+}
+
 }  // namespace chromeos