Add a helper function to query whether parental control logging is on for the current chrome user

base/win/metro.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/win/metro.h"
 
 #include "base/string_util.h"
+#include "base/win/scoped_comptr.h"
+#include "base/win/windows_version.h"
 
 namespace base {
 namespace win {
 
 const char kActivateApplication[] = "ActivateApplication";
 
 HMODULE GetMetroModule() {
   const HMODULE kUninitialized = reinterpret_cast<HMODULE>(1);
   static HMODULE metro_module = kUninitialized;
 
@@ skipping 58 matching lines @@
   base::wcslcpy(dest, src.c_str(), dest_size);
   return dest;
 }
 
 bool IsTouchEnabled() {
   int value = GetSystemMetrics(SM_DIGITIZER);
   return value & (NID_READY | NID_INTEGRATED_TOUCH) ==
              (NID_READY | NID_INTEGRATED_TOUCH);
 }
 
+bool IsParentalControlActivityLoggingOn() {
+  // Query this info on Windows 8 and above.
+  if (base::win::GetVersion() < base::win::VERSION_WIN8)
+    return false;
+
+  static bool parental_control_logging_required = false;

[jar (doing other things), 2012/07/11 21:07:06]

Is this only called on the UI thread, so that you don't have to worry about
races?  You could DCHECK() it... or at least put a comment in the header. 
DCHECK() is probably better.

[ananta, 2012/07/11 21:13:02]

Added a comment in the header.

+  static bool parental_control_status_determined = false;
+
+  if (parental_control_status_determined)
+    return parental_control_logging_required;
+
+  parental_control_status_determined = true;
+
+  ScopedComPtr<IWindowsParentalControlsCore> parent_controls;
+  HRESULT hr = parent_controls.CreateInstance(
+      __uuidof(WindowsParentalControls));
+  if (FAILED(hr))
+    return false;
+
+  ScopedComPtr<IWPCSettings> settings;
+  hr = parent_controls->GetUserSettings(NULL, settings.Receive());
+  if (FAILED(hr))
+    return false;
+
+  unsigned long restrictions = 0;
+  settings->GetRestrictions(&restrictions);
+
+  parental_control_logging_required =
+      (restrictions & WPCFLAG_LOGGING_REQUIRED == WPCFLAG_LOGGING_REQUIRED);

[jar (doing other things), 2012/07/11 21:07:06]

I think you need to move the trailing paren, or this will always be false:

(restrictions & WPCFLAG_LOGGING_REQUIRED) == WPCFLAG_LOGGING_REQUIRED;

[ananta, 2012/07/11 21:13:02]

Done.

+  return parental_control_logging_required;
+}
+
 }  // namespace win
 }  // namespace base