ADHOC/replicant-frameworks_native
3
0
Fork 0
Code Issues 6 Pull Requests Releases Wiki Activity

surfaceflinger: Use landscape resolution for default dpi

Browse Source 
When HWC doesn't provide DPI values for a display, we pick a default
DPI based on resolution. The intent was that 1080p and higher displays
would get XHIGH density, and lower resolutions would get TV density.
In KK (and possibly forever) we had a bug that we'd always use TV
density. That was fixed in L, but that fix exposed a pre-existing bug
that we always used the display's height in its native orientation,
rather than in landscape orientation. So an 800x1280 tablet like N7v1
started getting XHIGH density instead of the intended TV density.

Bug: 17461633
Change-Id: Ia57fa49e61f36bdda63ce283ef62c9953297222c
This commit is contained in:
Jesse Hall 2014-09-28 12:14:12 -07:00
parent 71e351d96b
commit bacc28ef1d
1 changed files with 13 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
services/surfaceflinger/DisplayHardware/HWComposer.cpp
View File

@ -314,9 +314,17 @@ void HWComposer::hotplug(int disp, int connected) {
mEventHandler.onHotplugReceived(disp, bool(connected)); mEventHandler.onHotplugReceived(disp, bool(connected));
} }
static float getDefaultDensity(uint32_t height) { static float getDefaultDensity(uint32_t width, uint32_t height) {
if (height >= 1080) return ACONFIGURATION_DENSITY_XHIGH; // Default density is based on TVs: 1080p displays get XHIGH density,
else return ACONFIGURATION_DENSITY_TV; // lower-resolution displays get TV density. Maybe eventually we'll need
// to update it for 4K displays, though hopefully those just report
// accurate DPI information to begin with. This is also used for virtual
// displays and even primary displays with older hwcomposers, so be
// careful about orientation.
uint32_t h = width < height ? width : height;
if (h >= 1080) return ACONFIGURATION_DENSITY_XHIGH;
else return ACONFIGURATION_DENSITY_TV;
} }
static const uint32_t DISPLAY_ATTRIBUTES[] = { static const uint32_t DISPLAY_ATTRIBUTES[] = {
@ -383,7 +391,7 @@ status_t HWComposer::queryDisplayProperties(int disp) {
} }
if (config.xdpi == 0.0f || config.ydpi == 0.0f) { if (config.xdpi == 0.0f || config.ydpi == 0.0f) {
float dpi = getDefaultDensity(config.height); float dpi = getDefaultDensity(config.width, config.height);
config.xdpi = dpi; config.xdpi = dpi;
config.ydpi = dpi; config.ydpi = dpi;
} }
@ -408,7 +416,7 @@ status_t HWComposer::setVirtualDisplayProperties(int32_t id,
DisplayConfig& config = mDisplayData[id].configs.editItemAt(configId); DisplayConfig& config = mDisplayData[id].configs.editItemAt(configId);
config.width = w; config.width = w;
config.height = h; config.height = h;
config.xdpi = config.ydpi = getDefaultDensity(h); config.xdpi = config.ydpi = getDefaultDensity(w, h);
return NO_ERROR; return NO_ERROR;
} }