Unify the Windows Parental Controls Platform Caching

chrome/browser/chrome_browser_main_win.cc

Index: chrome/browser/chrome_browser_main_win.cc
diff --git a/chrome/browser/chrome_browser_main_win.cc b/chrome/browser/chrome_browser_main_win.cc
index 822cd3c6b057a9a45ce7a1fb211f7cfacbeceae8..1cff812acf483755b21862cec6ce8bcb998d66b9 100644
--- a/chrome/browser/chrome_browser_main_win.cc
+++ b/chrome/browser/chrome_browser_main_win.cc
@@ -211,7 +211,8 @@ void ChromeBrowserMainPartsWin::PreMainMessageLoopStart() {
     InitializeWindowProcExceptions();
   }
 
-  IncognitoModePrefs::InitializePlatformParentalControls();
+  // Prime the parental controls cache on Windows.
+  IncognitoModePrefs::ArePlatformParentalControlsEnabled();

[grt (UTC plus 2), 2015/03/04 03:59:39]

ignore_result(...) to make this more explicit (and #include "base/macros.h").

[gab, 2015/03/04 16:25:23]

That makes sense to me, also use WARN_UNUSED_RESULT on the method declaration to
enforce this call site to explicitly do ignore_result().

[robliao, 2015/03/04 18:47:44]

Done.
FYI, WARN_UNUSED_RESULT only applies to gcc.

[gab, 2015/03/04 19:21:20]

Right, at least it gives it waterfall coverage (except in win-only code, but
better than nothing).

 }
 
 int ChromeBrowserMainPartsWin::PreCreateThreads() {