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

Unified Diff: ash/display/display_info.h

Issue 161413002: Revert of Read compositor VSync information from platform, when possible (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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 | « ash/display/display_controller.cc ('k') | ash/display/display_info.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/display/display_info.h
diff --git a/ash/display/display_info.h b/ash/display/display_info.h
index ccb989a15cf5f8685e806fe792c3984a3af0bcac..ca6f270a9e66163c1dfbcdc54c7f915988731e27 100644
--- a/ash/display/display_info.h
+++ b/ash/display/display_info.h
@@ -16,18 +16,13 @@
namespace ash {
namespace internal {
-// A struct that represents the display's mode info.
-struct ASH_EXPORT DisplayMode {
- DisplayMode();
- DisplayMode(const gfx::Size& size,
- float refresh_rate,
- bool interlaced,
- bool native);
+// A struct that represents the display's resolution and
+// interlaced info.
+struct ASH_EXPORT Resolution {
+ Resolution(const gfx::Size& size, bool interlaced);
- gfx::Size size; // Physical pixel size of the display.
- float refresh_rate; // Refresh rate of the display, in Hz.
- bool interlaced; // True if mode is interlaced.
- bool native; // True if mode is native mode of the display.
+ gfx::Size size;
+ bool interlaced;
};
// DisplayInfo contains metadata for each display. This is used to
@@ -52,7 +47,7 @@
// 270 degrees (to the 'l'eft).
// - ui-scale is floating value, e.g. @1.5 or @1.25.
// - |resolution list| is the list of size that is given in
- // |width x height [% refresh_rate]| separated by '|'.
+ // |width x height| separated by '|'.
//
// A couple of examples:
// "100x100"
@@ -68,9 +63,9 @@
// "10+20-300x200/u@1.5"
// 300x200 window at 10,20 origin. 1x device scale factor.
// no overscan. flipped upside-down (180 degree) and 1.5 ui scale.
- // "200x100#300x200|200x100%59.0|100x100%60"
+ // "200x100#300x200|200x100|100x100"
// 200x100 window at 0,0 origin, with 3 possible resolutions,
- // 300x200, 200x100 at 59 Hz, and 100x100 at 60 Hz.
+ // 300x200, 200x100 and 100x100.
static DisplayInfo CreateFromSpec(const std::string& spec);
// Creates a DisplayInfo from string spec using given |id|.
@@ -147,19 +142,19 @@
void set_native(bool native) { native_ = native; }
bool native() const { return native_; }
- const std::vector<DisplayMode>& display_modes() const {
- return display_modes_;
+ const std::vector<Resolution>& resolutions() const {
+ return resolutions_;
}
- void set_display_modes(std::vector<DisplayMode>& display_modes) {
- display_modes_.swap(display_modes);
+ void set_resolutions(std::vector<Resolution>& resolution) {
+ resolutions_.swap(resolution);
}
- // Returns a string representation of the DisplayInfo, excluding display
- // modes.
+ // Returns a string representation of the DisplayInfo
+ // excluding resolutions.
std::string ToString() const;
- // Returns a string representation of the DisplayInfo, including display
- // modes.
+ // Returns a string representation of the DisplayInfo
+ // including resolutions.
std::string ToFullString() const;
private:
@@ -193,8 +188,8 @@
// True if this comes from native platform (DisplayChangeObserver).
bool native_;
- // The list of modes supported by this display.
- std::vector<DisplayMode> display_modes_;
+ // The list of resolutions supported by this display.
+ std::vector<Resolution> resolutions_;
};
} // namespace internal
« no previous file with comments | « ash/display/display_controller.cc ('k') | ash/display/display_info.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698