Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1487)

Side by Side Diff: remoting/host/forward_process_stats_agent.h

Issue 2775983003: [Chromoting] Retrieve process resource usage (ProcessStats) and its tests (Closed)
Patch Set: Resolve review comments Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef REMOTING_HOST_FORWARD_PROCESS_STATS_AGENT_H_
6 #define REMOTING_HOST_FORWARD_PROCESS_STATS_AGENT_H_
7
8 #include "base/threading/thread_checker.h"
9 #include "remoting/host/process_stats_agent.h"
10 #include "remoting/proto/process_stats.pb.h"
11
12 namespace remoting {
13
14 // ProcessStatsAgent implementation that returns stats passed to
15 // OnProcessStats(). Used to collect stats from processes other than the current
16 // one. All public functions, not including constructor and destructor, of the
17 // object of ForwardProcessStatsAgent are expected to be called in a same
18 // thread.
19 class ForwardProcessStatsAgent final : public ProcessStatsAgent {
20 public:
21 ForwardProcessStatsAgent();
22 ~ForwardProcessStatsAgent() override;
23
24 void OnProcessStats(const protocol::ProcessResourceUsage& usage);
25
26 // ProcessStatsAgent implementation
joedow 2017/04/25 15:57:28 nit: add period
Hzj_jie 2017/04/25 22:19:19 Done.
27 protocol::ProcessResourceUsage GetResourceUsage() override;
28
29 private:
30 base::ThreadChecker thread_checker_;
31 protocol::ProcessResourceUsage usage_;
32 };
33
34 } // namespace remoting
35
36 #endif // REMOTING_HOST_FORWARD_PROCESS_STATS_AGENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698