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

Unified Diff: components/arc/power/arc_power_bridge.cc

Issue 2599673005: arc: Use GET_INTERFACE_FOR_METHOD macro (Closed)
Patch Set: Addressed feedback Created 3 years, 11 months 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: components/arc/power/arc_power_bridge.cc
diff --git a/components/arc/power/arc_power_bridge.cc b/components/arc/power/arc_power_bridge.cc
index c49d0b5fc4649a4cc98f05bee39fc05e646d71c0..5a17359d5e503b7db48d8e06c5edbdd99aaa6e6d 100644
--- a/components/arc/power/arc_power_bridge.cc
+++ b/components/arc/power/arc_power_bridge.cc
@@ -15,12 +15,6 @@
namespace arc {
-namespace {
-
-constexpr uint32_t kMinVersionForSetInteractive = 1;
-
-} // namespace
-
ArcPowerBridge::ArcPowerBridge(ArcBridgeService* bridge_service)
: ArcService(bridge_service), binding_(this) {
arc_bridge_service()->power()->AddObserver(this);
@@ -33,7 +27,7 @@ ArcPowerBridge::~ArcPowerBridge() {
void ArcPowerBridge::OnInstanceReady() {
mojom::PowerInstance* power_instance =
- arc_bridge_service()->power()->GetInstanceForMethod("Init");
+ ARC_GET_INSTANCE_FOR_METHOD(arc_bridge_service()->power(), Init);
DCHECK(power_instance);
power_instance->Init(binding_.CreateInterfacePtrAndBind());
ash::Shell::GetInstance()->display_configurator()->AddObserver(this);
@@ -46,9 +40,8 @@ void ArcPowerBridge::OnInstanceClosed() {
void ArcPowerBridge::OnPowerStateChanged(
chromeos::DisplayPowerState power_state) {
- mojom::PowerInstance* power_instance =
- arc_bridge_service()->power()->GetInstanceForMethod(
- "SetInteractive", kMinVersionForSetInteractive);
+ mojom::PowerInstance* power_instance = ARC_GET_INSTANCE_FOR_METHOD(
+ arc_bridge_service()->power(), SetInteractive);
if (!power_instance)
return;
« no previous file with comments | « components/arc/obb_mounter/arc_obb_mounter_bridge.cc ('k') | components/arc/storage_manager/arc_storage_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698