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

Add a helper function to query whether parental control logging is on for the current chrome user
to base\win\metro.

BUG=133560
R=cpu
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=146214

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

https://chromiumcodereview.appspot.com/10693133/diff/5002/base/win/metro.cc
File base/win/metro.cc (right):

https://chromiumcodereview.appspot.com/10693133/diff/5002/base/win/metro.cc#n...
base/win/metro.cc:93: static bool parental_control_logging_required = false;
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.

https://chromiumcodereview.appspot.com/10693133/diff/5002/base/win/metro.cc#n...
base/win/metro.cc:116: (restrictions & WPCFLAG_LOGGING_REQUIRED ==
WPCFLAG_LOGGING_REQUIRED);
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]

https://chromiumcodereview.appspot.com/10693133/diff/5002/base/win/metro.cc
File base/win/metro.cc (right):

https://chromiumcodereview.appspot.com/10693133/diff/5002/base/win/metro.cc#n...
base/win/metro.cc:93: static bool parental_control_logging_required = false;
On 2012/07/11 21:07:06, jar wrote:
> 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.

Added a comment in the header.

https://chromiumcodereview.appspot.com/10693133/diff/5002/base/win/metro.cc#n...
base/win/metro.cc:116: (restrictions & WPCFLAG_LOGGING_REQUIRED ==
WPCFLAG_LOGGING_REQUIRED);
On 2012/07/11 21:07:06, jar wrote:
> I think you need to move the trailing paren, or this will always be false:
> 
> (restrictions & WPCFLAG_LOGGING_REQUIRED) == WPCFLAG_LOGGING_REQUIRED;

Done.

[jar (doing other things), 2012-07-11 21:19:49]

lgtm