From 5cd8159cef1ad353f10d5027d08adfbb30d7cc6f Mon Sep 17 00:00:00 2001 From: Anay Wadhera Date: Mon, 14 Oct 2024 16:08:29 -0700 Subject: [PATCH] Remove GetOsPermissionStatus from WifiAdapter PiperOrigin-RevId: 685866926 --- sharing/internal/api/mock_wifi_adapter.h | 1 - sharing/internal/api/wifi_adapter.h | 3 --- sharing/internal/test/fake_wifi_adapter.h | 5 ----- sharing/internal/test/fake_wifi_adapter_test.cc | 6 ------ 4 files changed, 15 deletions(-) diff --git a/sharing/internal/api/mock_wifi_adapter.h b/sharing/internal/api/mock_wifi_adapter.h index 98700b61..9431ce37 100644 --- a/sharing/internal/api/mock_wifi_adapter.h +++ b/sharing/internal/api/mock_wifi_adapter.h @@ -35,7 +35,6 @@ class MockWifiAdapter : public nearby::sharing::api::WifiAdapter { MOCK_METHOD(bool, IsPresent, (), (const, override)); MOCK_METHOD(bool, IsPowered, (), (const, override)); - MOCK_METHOD(PermissionStatus, GetOsPermissionStatus, (), (const, override)); MOCK_METHOD(void, SetPowered, (bool powered, std::function success_callback, std::function error_callback), diff --git a/sharing/internal/api/wifi_adapter.h b/sharing/internal/api/wifi_adapter.h index 4f9d1321..61867373 100644 --- a/sharing/internal/api/wifi_adapter.h +++ b/sharing/internal/api/wifi_adapter.h @@ -59,9 +59,6 @@ class WifiAdapter { // Indicates whether the adapter radio is powered. virtual bool IsPowered() const = 0; - // Returns the status of the browser's Wi-Fi permission status. - virtual PermissionStatus GetOsPermissionStatus() const = 0; - // Requests a change to the adapter radio power. Setting `powered` to true // will turn on the radio and false will turn it off. On success, // `success_callback` will be called. On failure, `error_callback` will be diff --git a/sharing/internal/test/fake_wifi_adapter.h b/sharing/internal/test/fake_wifi_adapter.h index 9f1dc9ee..24c5f3a8 100644 --- a/sharing/internal/test/fake_wifi_adapter.h +++ b/sharing/internal/test/fake_wifi_adapter.h @@ -46,11 +46,6 @@ class FakeWifiAdapter : public sharing::api::WifiAdapter { return is_powered_; } - sharing::api::WifiAdapter::PermissionStatus GetOsPermissionStatus() - const override { - return sharing::api::WifiAdapter::PermissionStatus::kAllowed; - } - void SetPowered(bool powered, std::function success_callback, std::function error_callback) override { success_callback(); diff --git a/sharing/internal/test/fake_wifi_adapter_test.cc b/sharing/internal/test/fake_wifi_adapter_test.cc index b4dfeed5..4032badc 100644 --- a/sharing/internal/test/fake_wifi_adapter_test.cc +++ b/sharing/internal/test/fake_wifi_adapter_test.cc @@ -34,12 +34,6 @@ TEST(FakeWifiAdapter, IsPoweredReturnsTrueByDefault) { EXPECT_TRUE(fake_wifi_adapter.IsPowered()); } -TEST(FakeWifiAdapter, GetOSPermissionStatusReturnsAllowedByDefault) { - FakeWifiAdapter fake_wifi_adapter; - EXPECT_EQ(fake_wifi_adapter.GetOsPermissionStatus(), - sharing::api::WifiAdapter::PermissionStatus::kAllowed); -} - TEST(FakeWifiAdapter, SetPoweredRunsSuccessCallback) { bool powered_on; std::function success_callback = [&powered_on]() {