win10: Fix various ui glitches

chrome/browser/ui/views/frame/browser_desktop_window_tree_host_win.cc

Index: chrome/browser/ui/views/frame/browser_desktop_window_tree_host_win.cc
diff --git a/chrome/browser/ui/views/frame/browser_desktop_window_tree_host_win.cc b/chrome/browser/ui/views/frame/browser_desktop_window_tree_host_win.cc
index ea2d19d6fe7ab8f06155bcb00fd77c658dc9834d..d190f80f60955d137ec3c230bd49476b667d5045 100644
--- a/chrome/browser/ui/views/frame/browser_desktop_window_tree_host_win.cc
+++ b/chrome/browser/ui/views/frame/browser_desktop_window_tree_host_win.cc
@@ -7,6 +7,7 @@
 #include <dwmapi.h>
 
 #include "base/process/process_handle.h"
+#include "base/win/windows_version.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/themes/theme_service.h"
 #include "chrome/browser/themes/theme_service_factory.h"
@@ -343,6 +344,12 @@ MARGINS BrowserDesktopWindowTreeHostWin::GetDWMFrameMargins() const {
       tabstrip_bounds = gfx::win::DIPToScreenRect(tabstrip_bounds);
       margins.cyTopHeight = tabstrip_bounds.bottom() + kDWMFrameTopOffset;
     }
+    if (base::win::GetVersion() >= base::win::VERSION_WIN10) {

[Peter Kasting, 2015/06/25 23:34:12]

Nit: I think this might be clearer if instead of doing this here, you added a
conditional to where we set these nonzero above:

    if (!IsMaximized() && !GetWidget()->IsFullscreen()) {
      margins.cyTopHeight = kClientEdgeThickness + 1;
      // On Windows 10, we don't draw our own window border, so don't extend the
      // nonclient area in for it.
      if (base::win::GetVersion() < base::win::VERSION_WIN10) {
        ...

[scottmg, 2015/06/26 18:29:32]

Done.

+      // No inset on Win10.
+      margins.cxLeftWidth = 0;
+      margins.cxRightWidth = 0;
+      margins.cyBottomHeight = 0;
+    }
   }
   return margins;
 }