Commit 804eeab2 authored by Alex Chau's avatar Alex Chau Committed by Commit Bot

Fix HardwareInfo value in FakeLocalDeviceInfoProvider

- The parameter order of manufacturer was flipped

Bug: 1009454
Change-Id: Ia2bdcf4acb9693a7deab083c535db3f474337aab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1866728
Auto-Submit: Alex Chau <alexchau@chromium.org>
Reviewed-by: default avatarMikel Astiz <mastiz@chromium.org>
Commit-Queue: Alex Chau <alexchau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#707388}
parent 0d702034
......@@ -412,7 +412,7 @@ TEST_F(SharingServiceTest, SendMessageToDeviceSuccess) {
ASSERT_TRUE(
sharing_message.ParseFromString(fake_gcm_driver_.message().payload));
EXPECT_EQ("id", sharing_message.sender_guid());
EXPECT_EQ("model Computer manufacturer",
EXPECT_EQ("manufacturer Computer model",
sharing_message.sender_device_name());
// Simulate ack message received by AckMessageHandler.
......
......@@ -17,7 +17,7 @@ FakeLocalDeviceInfoProvider::FakeLocalDeviceInfoProvider()
"user_agent",
sync_pb::SyncEnums_DeviceType_TYPE_LINUX,
"device_id",
base::SysInfo::HardwareInfo{"model", "manufacturer", "serial"},
base::SysInfo::HardwareInfo{"manufacturer", "model", "serial"},
/*last_updated_timestamp=*/base::Time::Now(),
/*send_tab_to_self_receiving_enabled=*/false,
/*sharing_info=*/base::nullopt) {}
......
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