[Win] Statically link SHGetStockIconInfo.

Remove the dynamic linking of SHGetStockIconInfo as the project no longer
supports Windows XP where this was necessary.

Bug: 986625
Change-Id: I89abb16f8089fc475a9930f4f9ea4ad4ea1a71fa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726765Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: James Forshaw <forshaw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#682989}

chrome/browser/ui/views/elevation_icon_setter.cc

@@ -31,17 +31,8 @@ SkBitmap GetElevationIcon() {
     return SkBitmap();
 
   SHSTOCKICONINFO icon_info = { sizeof(SHSTOCKICONINFO) };
-  typedef HRESULT (STDAPICALLTYPE *GetStockIconInfo)(SHSTOCKICONID,
-                                                     UINT,
-                                                     SHSTOCKICONINFO*);
-  // Even with the runtime guard above, we have to use GetProcAddress()
-  // here, because otherwise the loader will try to resolve the function
-  // address on startup, which will break on XP.
-  GetStockIconInfo func = reinterpret_cast<GetStockIconInfo>(
-      GetProcAddress(GetModuleHandle(L"shell32.dll"), "SHGetStockIconInfo"));
-  // TODO(pkasting): Run on a background thread since this call spins a nested
-  // message loop.
-  if (FAILED((*func)(SIID_SHIELD, SHGSI_ICON | SHGSI_SMALLICON, &icon_info)))
+  if (FAILED(SHGetStockIconInfo(SIID_SHIELD, SHGSI_ICON | SHGSI_SMALLICON,
+                                &icon_info)))
     return SkBitmap();
 
   SkBitmap icon = IconUtil::CreateSkBitmapFromHICON(