Use actual value of display scale factor from GetMonitorScaleFactor().

ui/display/win/screen_win.cc

Index: ui/display/win/screen_win.cc
diff --git a/ui/display/win/screen_win.cc b/ui/display/win/screen_win.cc
index 86d6be41cd4e4a33e8a7dcc09600930582fa9e3e..1d92283a7e3e37ea6f3afc367c216244b855de26 100644
--- a/ui/display/win/screen_win.cc
+++ b/ui/display/win/screen_win.cc
@@ -440,6 +440,12 @@ gfx::Rect ScreenWin::DIPToScreenRectInWindow(gfx::NativeView view,
   return DIPToScreenRect(hwnd, dip_rect);
 }
 
+float ScreenWin::GetScaleFactorForWindow(gfx::NativeView view) const {
+  HWND hwnd = view ? GetHWNDFromNativeView(view) : nullptr;
+  return GetMonitorScaleFactor(

[Andrey Kraynov, 2016/11/16 18:33:47]

Another possible and simple way to fix this issue is to handle WM_DPICHANGED in
ScreenWin::OnWndProc and update displays.
But I failed to get it there. Should ScreenWin::OnWndProc() function receive
WM_DPICHANGED event?

[sadrul, 2016/11/16 20:38:17]

I think that would make sense, yeah. Is that possible to do? Looks like it
currently processes only WM_DISPLAYCHAGE, but WM_DPICHANGED would be something
it should process too.

[robliao, 2016/11/16 20:56:17]

WM_DPICHANGED is fired when an HWND is associated with a different DPI. This is
principally for per-monitor DPI window changes when you drag a window from one
screen to the other but can happen when you change the DPI of the HWND's current
display.

The SingletonHwnd is bound to the primary display at (0,0) with (0,0) size. As a
result, it will only get WM_DPICHANGED for the primary display. If change the
DPI on a secondary display, you may not receive a WM_DPICHANGED.

Taking a step back, I'm seeing WM_DISPLAYCHANGED fired first and then
WM_DPICHANGED.
This ordering is important from Windows's point for view for the reasons
described in the bug, applications that are doing display tracking like we are
need to update their metrics first before firing the WM_DPICHANGED.

I'll loop this discussion into the bug, but the main question I have is how does
one get WM_DPICHANGED to fire before WM_DISPLAYCHANGED?

+      ::MonitorFromWindow(hwnd, MONITOR_DEFAULTTOPRIMARY));
+}
+
 void ScreenWin::UpdateFromDisplayInfos(
     const std::vector<DisplayInfo>& display_infos) {
   screen_win_displays_ = DisplayInfosToScreenWinDisplays(display_infos);
@@ -487,6 +493,7 @@ void ScreenWin::OnWndProc(HWND hwnd,
   if (message != WM_DISPLAYCHANGE)
     return;
 
+  LOG(ERROR) << "ScreenWin::OnWndProc(WM_DISPLAYCHANGE)";
   std::vector<display::Display> old_displays = GetAllDisplays();
   UpdateFromDisplayInfos(GetDisplayInfosFromSystem());
   change_notifier_.NotifyDisplaysChanged(old_displays, GetAllDisplays());