[Chromoting] Retrieve process resource usage (ProcessStats) and its tests

remoting/host/current_process_stats_agent.h

+// Copyright 2017 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 REMOTING_HOST_CURRENT_PROCESS_STATS_AGENT_H_
+#define REMOTING_HOST_CURRENT_PROCESS_STATS_AGENT_H_
+
+#include <memory>
+#include <string>
+
+#include "base/process/process_metrics.h"

[joedow, 2017/04/24 17:16:08]

forward declare 'base::ProcessMetrics' since you don't need the definition in
the header.

[Hzj_jie, 2017/04/25 00:17:53]

Done.

+#include "remoting/host/process_stats_agent.h"
+#include "remoting/proto/process_stats.pb.h"
+
+namespace remoting {
+
+// A component to report statistic data of current process.
+class CurrentProcessStatsAgent final : public ProcessStatsAgent {
+ public:
+  CurrentProcessStatsAgent(const std::string& process_name);

[joedow, 2017/04/24 17:16:08]

single param c'tor should be marked explicit.

[Hzj_jie, 2017/04/25 00:17:53]

Done.

+  ~CurrentProcessStatsAgent() override;
+
+  protocol::ProcessResourceUsage GetResourceUsage() override;

[joedow, 2017/04/24 17:16:08]

nit: comment the override "// ProcessStatsAgent interface."

[Hzj_jie, 2017/04/25 00:17:53]

Done.

+
+ private:
+  const std::string process_name_;
+  const std::unique_ptr<base::ProcessMetrics> metrics_;
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_CURRENT_PROCESS_STATS_AGENT_H_