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

Unified Diff: components/arc/common/power.mojom

Issue 2805433002: Adding ArcBridge interface for synchronizing brightness settings (Closed)
Patch Set: Clean things up Created 3 years, 8 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/common/power.mojom
diff --git a/components/arc/common/power.mojom b/components/arc/common/power.mojom
index 195c7c828e391bbe58138f3472b98c25ba54b0cf..949517a04b71366499289307b8a47e1da864f492 100644
--- a/components/arc/common/power.mojom
+++ b/components/arc/common/power.mojom
@@ -19,7 +19,7 @@ enum DisplayWakeLockType {
DIM = 1
};
-// Next method ID: 3
+// Next method ID: 4
interface PowerHost {
// Acquire and release wake locks on the host side.
OnAcquireDisplayWakeLock@0(DisplayWakeLockType type);
@@ -27,9 +27,13 @@ interface PowerHost {
// Checks if there is a display on.
[MinVersion=1] IsDisplayOn@2() => (bool is_on);
+
+ // Set screen brightness.
+ // |brightness| is of the range [0, 255]
+ [MinVersion=2] SetBrightness@3(int32 brightness);
Luis Héctor Chávez 2017/04/05 21:28:02 What happened to the request to make this |double
Daniel Erat 2017/04/05 21:47:21 agreed; "double percentage" is better.
yueli 2017/04/05 23:27:17 Done.
};
-// Next method ID: 4
+// Next method ID: 5
interface PowerInstance {
// Establishes full-duplex communication with the host.
Init@0(PowerHost host_ptr);
@@ -43,4 +47,8 @@ interface PowerInstance {
// Called when the system has just resumed.
[MinVersion=2] Resume@3();
+
+ // Update android brightness settings.
+ // |brightness| is of the range [0, 255]
+ [MinVersion=3] UpdateBrightness@4(int32 brightness);
};

Powered by Google App Engine
This is Rietveld 408576698