Avoid calling the GetSystemMetrics API in the renderer process on Windows to get the scrollbar metr…

chrome/browser/renderer_preferences_util.cc

Index: chrome/browser/renderer_preferences_util.cc
diff --git a/chrome/browser/renderer_preferences_util.cc b/chrome/browser/renderer_preferences_util.cc
index ce174ef34c6375bea36ef73df2423d1df201c96f..bc96ae931c9e4d3e4001235193db9bbc390d4fd2 100644
--- a/chrome/browser/renderer_preferences_util.cc
+++ b/chrome/browser/renderer_preferences_util.cc
@@ -34,6 +34,7 @@
 #if defined(OS_WIN)
 #include "base/win/win_util.h"
 #include "ui/gfx/platform_font_win.h"
+#include "ui/gfx/win/dpi.h"
 #endif
 
 namespace renderer_preferences_util {
@@ -145,6 +146,15 @@ void UpdateFromSystemSettings(content::RendererPreferences* prefs,
   prefs->message_font_family_name = metrics.lfMessageFont.lfFaceName;
   prefs->message_font_height = gfx::PlatformFontWin::GetFontSize(
       metrics.lfMessageFont);
+
+  prefs->vertical_scroll_bar_width_in_dips =
+      gfx::win::GetSystemMetricsInDIP(SM_CXVSCROLL);
+  prefs->horizontal_scroll_bar_height_in_dips =

[brucedawson, 2015/04/03 17:00:24]

When are SM_CYHSCROLL, SM_CYVSCROLL, and SM_CXHSCROLL used?

[ananta, 2015/04/03 19:17:03]

They are not used as of now. I added those just in case we need them in the
future.

+      gfx::win::GetSystemMetricsInDIP(SM_CYHSCROLL);
+  prefs->arrow_bitmap_height_vertical_scroll_bar_in_dips =
+      gfx::win::GetSystemMetricsInDIP(SM_CYVSCROLL);
+  prefs->arrow_bitmap_width_horizontal_scroll_bar_in_dips =
+      gfx::win::GetSystemMetricsInDIP(SM_CXHSCROLL);

[jam, 2015/04/03 15:08:19]

seems this should be done in content layer instead of chrome so that all content
based browsers get this. same for some of the other stuff above


the only code that should be in this method is setting preferences that need
stuff from the chrome layer (i.e. from prefs)

[ananta, 2015/04/03 19:17:03]

Moved this as well as the font metrics code above to the
RenderViewHostImpl::CreateRenderView function.

 #endif
 }