Commit ee94e023 authored by Yusuke Sato's avatar Yusuke Sato Committed by Commit Bot

arc: Don't set num_cores_disabled proto field

ARC container does not use the field and it will soon be removed from
proto definition.

BUG=b:142144019
BUG=b:143175953
TEST=try

Change-Id: Ieabdbace116d9c70c35a031f0aee3fe648bdcd6d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1874580
Commit-Queue: Yusuke Sato <yusukes@chromium.org>
Auto-Submit: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: default avatarYury Khmel <khmel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#708434}
parent b886fbb7
...@@ -98,7 +98,6 @@ class ArcContainerClientAdapter ...@@ -98,7 +98,6 @@ class ArcContainerClientAdapter
request.set_arc_custom_tabs_experiment(params.arc_custom_tabs_experiment); request.set_arc_custom_tabs_experiment(params.arc_custom_tabs_experiment);
request.set_arc_print_spooler_experiment( request.set_arc_print_spooler_experiment(
params.arc_print_spooler_experiment); params.arc_print_spooler_experiment);
request.set_num_cores_disabled(params.num_cores_disabled);
chromeos::SessionManagerClient::Get()->StartArcMiniContainer( chromeos::SessionManagerClient::Get()->StartArcMiniContainer(
request, std::move(callback)); request, std::move(callback));
......
...@@ -731,9 +731,6 @@ TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensity) { ...@@ -731,9 +731,6 @@ TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensity) {
EXPECT_EQ(240, chromeos::FakeSessionManagerClient::Get() EXPECT_EQ(240, chromeos::FakeSessionManagerClient::Get()
->last_start_arc_mini_container_request() ->last_start_arc_mini_container_request()
.lcd_density()); .lcd_density());
EXPECT_EQ(2u, chromeos::FakeSessionManagerClient::Get()
->last_start_arc_mini_container_request()
.num_cores_disabled());
} }
TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsync) { TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsync) {
...@@ -754,9 +751,6 @@ TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsync) { ...@@ -754,9 +751,6 @@ TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsync) {
EXPECT_EQ(240, chromeos::FakeSessionManagerClient::Get() EXPECT_EQ(240, chromeos::FakeSessionManagerClient::Get()
->last_start_arc_mini_container_request() ->last_start_arc_mini_container_request()
.lcd_density()); .lcd_density());
EXPECT_EQ(2u, chromeos::FakeSessionManagerClient::Get()
->last_start_arc_mini_container_request()
.num_cores_disabled());
} }
TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsyncReversedOrder) { TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsyncReversedOrder) {
...@@ -776,9 +770,6 @@ TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsyncReversedOrder) { ...@@ -776,9 +770,6 @@ TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsyncReversedOrder) {
EXPECT_EQ(240, chromeos::FakeSessionManagerClient::Get() EXPECT_EQ(240, chromeos::FakeSessionManagerClient::Get()
->last_start_arc_mini_container_request() ->last_start_arc_mini_container_request()
.lcd_density()); .lcd_density());
EXPECT_EQ(2u, chromeos::FakeSessionManagerClient::Get()
->last_start_arc_mini_container_request()
.num_cores_disabled());
} }
TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsyncCpuInfoEarly) { TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsyncCpuInfoEarly) {
...@@ -798,9 +789,6 @@ TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsyncCpuInfoEarly) { ...@@ -798,9 +789,6 @@ TEST_F(ArcSessionImplTest, StartArcMiniContainerWithDensityAsyncCpuInfoEarly) {
EXPECT_EQ(240, chromeos::FakeSessionManagerClient::Get() EXPECT_EQ(240, chromeos::FakeSessionManagerClient::Get()
->last_start_arc_mini_container_request() ->last_start_arc_mini_container_request()
.lcd_density()); .lcd_density());
EXPECT_EQ(2u, chromeos::FakeSessionManagerClient::Get()
->last_start_arc_mini_container_request()
.num_cores_disabled());
} }
TEST_F(ArcSessionImplTest, StopWhileWaitingForLcdDensity) { TEST_F(ArcSessionImplTest, StopWhileWaitingForLcdDensity) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment