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

Unified Diff: content/browser/memory/memory_coordinator_unittest.cc

Issue 2480203002: ui: Cleanup class/struct forward declarations (Closed)
Patch Set: Sync CL to position 430550 Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/memory/memory_coordinator_unittest.cc
diff --git a/content/browser/memory/memory_coordinator_unittest.cc b/content/browser/memory/memory_coordinator_unittest.cc
index eb81c9823a4ad553d627db80a7380765e4aac77b..d6cf7df5ddf3f899a64970946c5b42fa9dc0fdd8 100644
--- a/content/browser/memory/memory_coordinator_unittest.cc
+++ b/content/browser/memory/memory_coordinator_unittest.cc
@@ -72,9 +72,9 @@ class TestMemoryCoordinator : public MemoryCoordinator {
}
// Wrapper of MemoryCoordinator::SetMemoryState that also calls RunUntilIdle.
- bool SetMemoryState(
+ bool SetChildMemoryState(
int render_process_id, mojom::MemoryState memory_state) {
- bool result = MemoryCoordinator::SetMemoryState(
+ bool result = MemoryCoordinator::SetChildMemoryState(
render_process_id, memory_state);
RunUntilIdle();
return result;
@@ -102,7 +102,7 @@ TEST_F(MemoryCoordinatorTest, SetMemoryStateFailsInvalidState) {
TestMemoryCoordinator mc;
auto cmc1 = mc.CreateChildMemoryCoordinator(1);
- EXPECT_FALSE(mc.SetMemoryState(1, mojom::MemoryState::UNKNOWN));
+ EXPECT_FALSE(mc.SetChildMemoryState(1, mojom::MemoryState::UNKNOWN));
EXPECT_EQ(0, cmc1->on_state_change_calls());
}
@@ -110,7 +110,7 @@ TEST_F(MemoryCoordinatorTest, SetMemoryStateFailsInvalidRenderer) {
TestMemoryCoordinator mc;
auto cmc1 = mc.CreateChildMemoryCoordinator(1);
- EXPECT_FALSE(mc.SetMemoryState(2, mojom::MemoryState::THROTTLED));
+ EXPECT_FALSE(mc.SetChildMemoryState(2, mojom::MemoryState::THROTTLED));
EXPECT_EQ(0, cmc1->on_state_change_calls());
}
@@ -118,7 +118,7 @@ TEST_F(MemoryCoordinatorTest, SetMemoryStateNotDeliveredNop) {
TestMemoryCoordinator mc;
auto cmc1 = mc.CreateChildMemoryCoordinator(1);
- EXPECT_FALSE(mc.SetMemoryState(2, mojom::MemoryState::NORMAL));
+ EXPECT_FALSE(mc.SetChildMemoryState(2, mojom::MemoryState::NORMAL));
EXPECT_EQ(0, cmc1->on_state_change_calls());
}
@@ -127,11 +127,11 @@ TEST_F(MemoryCoordinatorTest, SetMemoryStateDelivered) {
auto cmc1 = mc.CreateChildMemoryCoordinator(1);
auto cmc2 = mc.CreateChildMemoryCoordinator(2);
- EXPECT_TRUE(mc.SetMemoryState(1, mojom::MemoryState::THROTTLED));
+ EXPECT_TRUE(mc.SetChildMemoryState(1, mojom::MemoryState::THROTTLED));
EXPECT_EQ(1, cmc1->on_state_change_calls());
EXPECT_EQ(mojom::MemoryState::THROTTLED, cmc1->state());
- EXPECT_TRUE(mc.SetMemoryState(2, mojom::MemoryState::SUSPENDED));
+ EXPECT_TRUE(mc.SetChildMemoryState(2, mojom::MemoryState::SUSPENDED));
EXPECT_EQ(1, cmc2->on_state_change_calls());
EXPECT_EQ(mojom::MemoryState::SUSPENDED, cmc2->state());
}
« no previous file with comments | « content/browser/memory/memory_coordinator_impl.cc ('k') | content/browser/renderer_host/clipboard_message_filter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698