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

Unified Diff: chromecast/public/avsettings.h

Issue 2384823002: Add SetVideoMode API to avsettings (Closed)
Patch Set: Addressed comments Created 4 years, 2 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromecast/public/avsettings.h
diff --git a/chromecast/public/avsettings.h b/chromecast/public/avsettings.h
index be12e2183c309a8e4e343073c9374a511652cd5f..4bd29c74f6b9bcc27bb5b86ed41f4a16775c6ff2 100644
--- a/chromecast/public/avsettings.h
+++ b/chromecast/public/avsettings.h
@@ -291,6 +291,22 @@ class AvSettings {
// Gets the current HDR output type.
virtual HdrOutputType GetHdrOutputType() = 0;
+
+ // Sets the HDMI video mode according to the given parameters:
+ // |allow_4k|: if false, the resolution set will not be a 4K resolution.
+ // |optimize_for_fps|: *Attempts* to pick a refresh rate optimal for the
+ // given content frame rate. |optimize_for_fps| is expressed as framerate
+ // * 100. I.e. 24hz -> 2400, 23.98hz -> 2398, etc. Values <= 0 are ignored.
+ //
+ // Returns:
+ // - true if HDMI video mode change is beginning. Caller should wait for
+ // SCREEN_INFO_CHANGED event for mode change to complete.
+ // - false if no HDMI video mode change has begun. This could be because
+ // HDMI is disconnected, or the current resolution is already good for the
+ // given parameters.
+ //
+ // Non-HDMI devices should return false.
+ virtual bool SetHdmiVideoMode(bool allow_4k, int optimize_for_fps) = 0;
};
} // namespace chromecast
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698