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"
+
 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::Bind(&FakeUpdateEngineClient::OnGetChannel,

[hashimoto, 2016/06/21 03:09:52]

This line does nothing.
If you want to run the callback asynchronously, it should be:

  base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE, base::Bind(callback,
<callback argument>));

[xiaoyinh(OOO Sep 11-29), 2016/06/22 18:25:40]

Thank you so much! Done.

+             weak_ptr_factory_.GetWeakPtr(), callback);
 }
 
 void FakeUpdateEngineClient::GetEolStatus(
-    const GetEolStatusCallback& callback) {}
+    const GetEolStatusCallback& callback) {
+  base::Bind(&FakeUpdateEngineClient::OnGetEolStatus,

[hashimoto, 2016/06/21 03:09:52]

ditto.

[xiaoyinh(OOO Sep 11-29), 2016/06/22 18:25:40]

Done.

+             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;
 }
 
 }  // namespace chromeos