Adding ArcBridge interface for synchronizing brightness settings

chromeos/dbus/power_manager_client.cc

Index: chromeos/dbus/power_manager_client.cc
diff --git a/chromeos/dbus/power_manager_client.cc b/chromeos/dbus/power_manager_client.cc
index 2ee1a195e42d04e356eff8206be6996c6852147d..54e37ee35a87cba9a4a12870b0e66e17ca36f7c0 100644
--- a/chromeos/dbus/power_manager_client.cc
+++ b/chromeos/dbus/power_manager_client.cc
@@ -186,6 +186,18 @@ class PowerManagerClientImpl : public PowerManagerClient {
                    weak_ptr_factory_.GetWeakPtr(), callback));
   }
 
+  void SyncScreenBrightness() override {
+    const GetScreenBrightnessPercentCallback& callback =
+        base::Bind(&PowerManagerClientImpl::NotifyBrightnessChanges,
+                   weak_ptr_factory_.GetWeakPtr());

[Muyuan, 2017/04/05 18:41:08]

You can inline callback into next line. Also there is no need for ref const
GetScreenBrightnessPercentCallback&

[yueli, 2017/04/05 19:59:46]

Done.

+    GetScreenBrightnessPercent(callback);
+  }
+
+  void NotifyBrightnessChanges(double percent) {
+    for (auto& observer : observers_)
+      observer.NotifyBrightnessChange(percent);
+  }
+
   void RequestStatusUpdate() override {
     POWER_LOG(USER) << "RequestStatusUpdate";
     dbus::MethodCall method_call(
@@ -467,8 +479,10 @@ class PowerManagerClientImpl : public PowerManagerClient {
     }
     POWER_LOG(DEBUG) << "Brightness changed to " << brightness_level
                      << ": user initiated " << user_initiated;
-    for (auto& observer : observers_)
+    for (auto& observer : observers_) {
       observer.BrightnessChanged(brightness_level, user_initiated);
+      observer.NotifyBrightnessChange(static_cast<double>(brightness_level));
+    }
   }
 
   void PeripheralBatteryStatusReceived(dbus::Signal* signal) {