Commit 6a8bb163 authored by Oleh Lamzin's avatar Oleh Lamzin Committed by Commit Bot

WilcoDtcSupportdBridge: move HandleEvent to private

Move WilcoDtcSupportdBridge::HandleEvent function to private section.

Bug:none
Test:existing unittests

Change-Id: I13982544accd0c924900941ecca6f83dc414e349
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1985987Reviewed-by: default avatarMaksim Ivanov <emaxx@chromium.org>
Commit-Queue: Oleh Lamzin <lamzin@google.com>
Cr-Commit-Position: refs/heads/master@{#728212}
parent ab46fc6c
......@@ -83,10 +83,6 @@ class WilcoDtcSupportdBridge final
: nullptr;
}
// wilco_dtc_supportd::mojom::WilcoDtcSupportdClient overrides.
void HandleEvent(
wilco_dtc_supportd::mojom::WilcoDtcSupportdEvent event) override;
private:
// Starts waiting until the wilco_dtc_supportd D-Bus service becomes available
// (or until this waiting fails).
......@@ -118,6 +114,8 @@ class WilcoDtcSupportdBridge final
mojo::ScopedHandle json_message,
SendWilcoDtcMessageToUiCallback callback) override;
void GetConfigurationData(GetConfigurationDataCallback callback) override;
void HandleEvent(
wilco_dtc_supportd::mojom::WilcoDtcSupportdEvent event) override;
std::unique_ptr<Delegate> delegate_;
......
......@@ -210,6 +210,11 @@ class WilcoDtcSupportdBridgeTest : public testing::Test {
return wilco_dtc_supportd_bridge_.get();
}
wilco_dtc_supportd::mojom::WilcoDtcSupportdClient*
wilco_dtc_supportd_client() {
return wilco_dtc_supportd_bridge_.get();
}
FakeWilcoDtcSupportdClient* wilco_dtc_supportd_dbus_client() {
WilcoDtcSupportdClient* const wilco_dtc_supportd_client =
WilcoDtcSupportdClient::Get();
......@@ -460,19 +465,19 @@ TEST_F(WilcoDtcSupportdBridgeTest, RetryCounterReset) {
// Test that the bridge calls the right method on the notification controller.
TEST_F(WilcoDtcSupportdBridgeTest, HandleEvent) {
EXPECT_CALL(*notification_controller(), ShowBatteryAuthNotification());
wilco_dtc_supportd_bridge()->HandleEvent(
wilco_dtc_supportd_client()->HandleEvent(
wilco_dtc_supportd::mojom::WilcoDtcSupportdEvent::kBatteryAuth);
EXPECT_CALL(*notification_controller(), ShowIncompatibleDockNotification());
wilco_dtc_supportd_bridge()->HandleEvent(
wilco_dtc_supportd_client()->HandleEvent(
wilco_dtc_supportd::mojom::WilcoDtcSupportdEvent::kIncompatibleDock);
EXPECT_CALL(*notification_controller(), ShowNonWilcoChargerNotification());
wilco_dtc_supportd_bridge()->HandleEvent(
wilco_dtc_supportd_client()->HandleEvent(
wilco_dtc_supportd::mojom::WilcoDtcSupportdEvent::kNonWilcoCharger);
EXPECT_CALL(*notification_controller(), ShowDockErrorNotification());
wilco_dtc_supportd_bridge()->HandleEvent(
wilco_dtc_supportd_client()->HandleEvent(
wilco_dtc_supportd::mojom::WilcoDtcSupportdEvent::kDockError);
}
......
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