NOT FOR SUBMIT - Windows Views HiDPI

ui/views/win/hwnd_message_handler.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/views/win/hwnd_message_handler.h"
 
 #include <dwmapi.h>
 #include <shellapi.h>
 
 #include "base/bind.h"
@@ skipping 502 matching lines @@
   }
 
   if (show_state) {
     if (wp.showCmd == SW_SHOWMAXIMIZED)
       *show_state = ui::SHOW_STATE_MAXIMIZED;
     else if (wp.showCmd == SW_SHOWMINIMIZED)
       *show_state = ui::SHOW_STATE_MINIMIZED;
     else
       *show_state = ui::SHOW_STATE_NORMAL;
   }
+
+  *bounds = ui::win::ScreenToDIPRect(*bounds);

[kevers, 2013/04/19 20:31:42]

As it stands, this change will break Win-Aura, which does the transformation in
the caller of this method.

[girard, 2013/06/03 22:31:55]

I've moved this up to the caller (NativeWidgetWin::GetWindowPlacement) but I
can't find the corresponding conversion in NativeWidgetAura. Where should I be
looking?

 }
 
 void HWNDMessageHandler::SetBounds(const gfx::Rect& bounds) {
   LONG style = GetWindowLong(hwnd(), GWL_STYLE);
   if (style & WS_MAXIMIZE)
     SetWindowLong(hwnd(), GWL_STYLE, style & ~WS_MAXIMIZE);
   SetWindowPos(hwnd(), NULL, bounds.x(), bounds.y(), bounds.width(),
                bounds.height(), SWP_NOACTIVATE | SWP_NOZORDER);
 }
 
@@ skipping 1575 matching lines @@
     DwmExtendFrameIntoClientArea(hwnd(), &m);
   }
   if (window_pos->flags & SWP_SHOWWINDOW)
     delegate_->HandleVisibilityChanged(true);
   else if (window_pos->flags & SWP_HIDEWINDOW)
     delegate_->HandleVisibilityChanged(false);
   SetMsgHandled(FALSE);
 }
 
 }  // namespace views