Adding ArcBridge interface for synchronizing brightness settings

components/arc/power/arc_power_bridge.h

Index: components/arc/power/arc_power_bridge.h
diff --git a/components/arc/power/arc_power_bridge.h b/components/arc/power/arc_power_bridge.h
index c459cffbb07d22b9066368ff692ff99dfba014c3..8d314e920bfceddf0af2ce2b521f2e6bcc96868e 100644
--- a/components/arc/power/arc_power_bridge.h
+++ b/components/arc/power/arc_power_bridge.h
@@ -41,13 +41,18 @@ class ArcPowerBridge : public ArcService,
   // DisplayConfigurator::Observer overrides.
   void OnPowerStateChanged(chromeos::DisplayPowerState power_state) override;
 
+  // PowerManagerClient::Observer overrides.
+  void BrightnessChanged(int level, bool user_initiated) override;
+
   // mojom::PowerHost overrides.
   void OnAcquireDisplayWakeLock(mojom::DisplayWakeLockType type) override;
   void OnReleaseDisplayWakeLock(mojom::DisplayWakeLockType type) override;
   void IsDisplayOn(const IsDisplayOnCallback& callback) override;
+  void SetBrightness(double brightness) override;

[Luis Héctor Chávez, 2017/04/05 23:55:28]

nit: s/double brightness/double percent/

 
  private:
   void ReleaseAllDisplayWakeLocks();
+  void UpdateBrightness(double percent);

[Daniel Erat, 2017/04/05 23:43:38]

nit: rename to UpdateAndroidBrightness to make it clearer what this actually
does

[Luis Héctor Chávez, 2017/04/05 23:55:28]

Same thing, "Screen" somewhere.

[yueli, 2017/04/06 01:12:37]

Done.

 
   mojo::Binding<mojom::PowerHost> binding_;
 
@@ -55,6 +60,8 @@ class ArcPowerBridge : public ArcService,
   // held by ARC.
   std::multimap<mojom::DisplayWakeLockType, int> wake_locks_;
 
+  base::WeakPtrFactory<ArcPowerBridge> weak_ptr_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(ArcPowerBridge);
 };