commit 0a28ed6e3747062d54118429e37503758d8e50ad
author John Abd-El-Malek <jam@chromium.org> Tue Nov 04 20:19:09 2014 -0800
committer John Abd-El-Malek <jam@chromium.org> Tue Nov 04 20:19:09 2014 -0800
tree 6339ff722db842274a7f09788942ccd2413fcaea
parent 72988f3bce80c9d3585a5a4107cd4cb42d3639ef

Reapply win gn fixes from
https://codereview.chromium.org/694353003/
https://codereview.chromium.org/697183003/
https://codereview.chromium.org/700763002/

that were undone by
https://codereview.chromium.org/693943003

R=scottmg@chromium.org

Review URL: https://codereview.chromium.org/701953004

win/win_util.cc

diff --git a/win/win_util.cc b/win/win_util.cc
index f46242d..21b78fa 100644
--- a/win/win_util.cc
+++ b/win/win_util.cc
@@ -119,14 +119,12 @@
 
 static bool g_crash_on_process_detach = false;
 
-void GetNonClientMetrics(NONCLIENTMETRICS_XP* metrics) {
+void GetNonClientMetrics(NONCLIENTMETRICS* metrics) {
   DCHECK(metrics);
   metrics->cbSize = sizeof(*metrics);
-  const bool success = !!SystemParametersInfo(
-      SPI_GETNONCLIENTMETRICS,
-      metrics->cbSize,
-      reinterpret_cast<NONCLIENTMETRICS*>(metrics),
-      0);
+  const bool success =
+      !!SystemParametersInfo(
+          SPI_GETNONCLIENTMETRICS, metrics->cbSize, metrics, 0);
   DCHECK(success);
 }
 

win/win_util.h

diff --git a/win/win_util.h b/win/win_util.h
index 8513f62..9439597 100644
--- a/win/win_util.h
+++ b/win/win_util.h
@@ -33,30 +33,10 @@
 struct _tagpropertykey;
 typedef _tagpropertykey PROPERTYKEY;
 
-// This is the same as NONCLIENTMETRICS except that the
-// unused member |iPaddedBorderWidth| has been removed.
-struct NONCLIENTMETRICS_XP {
-    UINT    cbSize;
-    int     iBorderWidth;
-    int     iScrollWidth;
-    int     iScrollHeight;
-    int     iCaptionWidth;
-    int     iCaptionHeight;
-    LOGFONTW lfCaptionFont;
-    int     iSmCaptionWidth;
-    int     iSmCaptionHeight;
-    LOGFONTW lfSmCaptionFont;
-    int     iMenuWidth;
-    int     iMenuHeight;
-    LOGFONTW lfMenuFont;
-    LOGFONTW lfStatusFont;
-    LOGFONTW lfMessageFont;
-};
-
 namespace base {
 namespace win {
 
-BASE_EXPORT void GetNonClientMetrics(NONCLIENTMETRICS_XP* metrics);
+BASE_EXPORT void GetNonClientMetrics(NONCLIENTMETRICS* metrics);
 
 // Returns the string representing the current user sid.
 BASE_EXPORT bool GetUserSidString(std::wstring* user_sid);