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

Unified Diff: ash/display/display_info.h

Issue 417113012: Introduce user customization of external HighDPI mode for 4K monitor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 6 years, 5 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: ash/display/display_info.h
diff --git a/ash/display/display_info.h b/ash/display/display_info.h
index a8bca41e4d7bf7371034a1183728d51378016572..627e8695dfaf2b03fdfbfbe355a6732c5f98c5ea 100644
--- a/ash/display/display_info.h
+++ b/ash/display/display_info.h
@@ -24,10 +24,18 @@ struct ASH_EXPORT DisplayMode {
bool interlaced,
bool native);
+ // Returns the size in DIP which isvisible to the user.
+ gfx::Size GetSizeInDIP() const;
+
+ bool operator==(const DisplayMode& other) const;
oshima 2014/07/31 02:16:52 can you add comment that this is necessary for stl
Jun Mukai 2014/07/31 04:57:52 This was added for options handler (check if an op
oshima 2014/07/31 18:52:37 I'm personally fine with this as you used this in
+
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.
+ float ui_scale; // The UI scale factor of the mode.
+ // TODO(mukai|oshima): merge to device_scale_factor.
oshima 2014/07/31 02:16:52 should we?
Jun Mukai 2014/07/31 04:57:52 removed because you're suspicious.
+ float device_scale_factor; // The device scale factor of the mode.
};
// DisplayInfo contains metadata for each display. This is used to

Powered by Google App Engine
This is Rietveld 408576698