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

Side by Side Diff: services/resource_coordinator/memory/coordinator/coordinator_impl_unittest.cc

Issue 2694083005: memory-infra: Finish moving memory_infra from TracingController (Closed)
Patch Set: review Created 3 years, 10 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
1 // Copyright 2017 The Chromium Authors. All rights reserved. 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "services/resource_coordinator/memory/coordinator/coordinator_impl.h" 5 #include "services/resource_coordinator/memory/coordinator/coordinator_impl.h"
6 #include "base/bind_helpers.h" 6 #include "base/bind_helpers.h"
7 #include "base/callback_forward.h" 7 #include "base/callback_forward.h"
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/run_loop.h" 10 #include "base/run_loop.h"
11 #include "base/trace_event/memory_dump_request_args.h" 11 #include "base/trace_event/memory_dump_request_args.h"
12 #include "mojo/public/cpp/bindings/binding.h" 12 #include "mojo/public/cpp/bindings/binding.h"
13 #include "mojo/public/cpp/bindings/interface_request.h" 13 #include "mojo/public/cpp/bindings/interface_request.h"
14 #include "services/resource_coordinator/public/interfaces/memory/memory_instrume ntation.mojom.h" 14 #include "services/resource_coordinator/public/interfaces/memory/memory_instrume ntation.mojom.h"
15 #include "testing/gtest/include/gtest/gtest.h" 15 #include "testing/gtest/include/gtest/gtest.h"
16 16
17 namespace memory_instrumentation { 17 namespace memory_instrumentation {
18 18
19 class CoordinatorImplTest : public testing::Test { 19 class CoordinatorImplTest : public testing::Test {
20 public: 20 public:
21 CoordinatorImplTest() {} 21 CoordinatorImplTest() {}
22 void SetUp() override { 22 void SetUp() override {
23 dump_response_args_ = {static_cast<uint64_t>(-1), false}; 23 dump_response_args_ = {static_cast<uint64_t>(-1), false};
24 coordinator_.reset(new CoordinatorImpl());
25 coordinator_->InitializeForTest(base::ThreadTaskRunnerHandle::Get().get());
24 } 26 }
25 27
28 void TearDown() override { coordinator_.reset(); }
29
26 void RegisterProcessLocalDumpManager( 30 void RegisterProcessLocalDumpManager(
27 mojom::ProcessLocalDumpManagerPtr process_manager) { 31 mojom::ProcessLocalDumpManagerPtr process_manager) {
28 base::ThreadTaskRunnerHandle::Get()->PostTask( 32 base::ThreadTaskRunnerHandle::Get()->PostTask(
29 FROM_HERE, base::Bind(&CoordinatorImpl::RegisterProcessLocalDumpManager, 33 FROM_HERE, base::Bind(&CoordinatorImpl::RegisterProcessLocalDumpManager,
30 base::Unretained(&coordinator_), 34 base::Unretained(coordinator_.get()),
31 base::Passed(&process_manager))); 35 base::Passed(&process_manager)));
32 } 36 }
33 37
34 void RequestGlobalMemoryDump(base::trace_event::MemoryDumpRequestArgs args, 38 void RequestGlobalMemoryDump(base::trace_event::MemoryDumpRequestArgs args,
35 base::Closure closure) { 39 base::Closure closure) {
36 base::ThreadTaskRunnerHandle::Get()->PostTask( 40 base::ThreadTaskRunnerHandle::Get()->PostTask(
37 FROM_HERE, 41 FROM_HERE,
38 base::Bind(&CoordinatorImpl::RequestGlobalMemoryDump, 42 base::Bind(&CoordinatorImpl::RequestGlobalMemoryDump,
39 base::Unretained(&coordinator_), args, 43 base::Unretained(coordinator_.get()), args,
40 base::Bind(&CoordinatorImplTest::OnGlobalMemoryDumpResponse, 44 base::Bind(&CoordinatorImplTest::OnGlobalMemoryDumpResponse,
41 base::Unretained(this), closure))); 45 base::Unretained(this), closure)));
42 } 46 }
43 47
44 void OnGlobalMemoryDumpResponse(base::Closure closure, 48 void OnGlobalMemoryDumpResponse(base::Closure closure,
45 uint64_t dump_guid, 49 uint64_t dump_guid,
46 bool success) { 50 bool success) {
47 dump_response_args_ = {dump_guid, success}; 51 dump_response_args_ = {dump_guid, success};
48 closure.Run(); 52 closure.Run();
49 } 53 }
50 54
51 protected: 55 protected:
52 struct DumpResponseArgs { 56 struct DumpResponseArgs {
53 uint64_t dump_guid; 57 uint64_t dump_guid;
54 bool success; 58 bool success;
55 }; 59 };
56 60
57 DumpResponseArgs dump_response_args_; 61 DumpResponseArgs dump_response_args_;
58 62
59 private: 63 private:
60 CoordinatorImpl coordinator_; 64 std::unique_ptr<CoordinatorImpl> coordinator_;
61 base::MessageLoop message_loop_; 65 base::MessageLoop message_loop_;
62 }; 66 };
63 67
64 class MockDumpManager : mojom::ProcessLocalDumpManager { 68 class MockDumpManager : mojom::ProcessLocalDumpManager {
65 public: 69 public:
66 MockDumpManager(CoordinatorImplTest* test_coordinator, int expected_calls) 70 MockDumpManager(CoordinatorImplTest* test_coordinator, int expected_calls)
67 : binding_(this), expected_calls_(expected_calls) { 71 : binding_(this), expected_calls_(expected_calls) {
68 // Register to the coordinator. 72 // Register to the coordinator.
69 mojom::ProcessLocalDumpManagerPtr process_manager; 73 mojom::ProcessLocalDumpManagerPtr process_manager;
70 binding_.Bind(mojo::MakeRequest(&process_manager)); 74 binding_.Bind(mojo::MakeRequest(&process_manager));
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 base::ThreadTaskRunnerHandle::Get()->PostTask( 133 base::ThreadTaskRunnerHandle::Get()->PostTask(
130 FROM_HERE, base::Bind([](std::unique_ptr<MockDumpManager> dm) {}, 134 FROM_HERE, base::Bind([](std::unique_ptr<MockDumpManager> dm) {},
131 base::Passed(&dump_manager_2))); 135 base::Passed(&dump_manager_2)));
132 136
133 run_loop.Run(); 137 run_loop.Run();
134 138
135 EXPECT_EQ(static_cast<uint64_t>(3456), dump_response_args_.dump_guid); 139 EXPECT_EQ(static_cast<uint64_t>(3456), dump_response_args_.dump_guid);
136 EXPECT_FALSE(dump_response_args_.success); 140 EXPECT_FALSE(dump_response_args_.success);
137 } 141 }
138 } // namespace memory_instrumentation 142 } // namespace memory_instrumentation
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698