Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1102)

Unified Diff: components/breakpad/breakpad_mac.mm

Issue 27031002: Move checking of configured policy on win to breakpad client (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: updates Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/breakpad/breakpad_client.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/breakpad/breakpad_mac.mm
diff --git a/components/breakpad/breakpad_mac.mm b/components/breakpad/breakpad_mac.mm
index 0f6c09761b8f882ef4c287591c10600ea74e74c6..e109f9fb2d9b49e0d594898f39d7c439ce011314 100644
--- a/components/breakpad/breakpad_mac.mm
+++ b/components/breakpad/breakpad_mac.mm
@@ -166,10 +166,7 @@ void InitCrashReporter() {
if (is_browser) {
// Since the configuration management infrastructure is possibly not
// initialized when this code runs, read the policy preference directly.
- if (GetBreakpadClient()->ReportingIsEnforcedByPolicy()) {
- // Controlled by configuration manangement.
- enable_breakpad = true;
- } else {
+ if (!GetBreakpadClient()->ReportingIsEnforcedByPolicy(&enable_breakpad)) {
// Controlled by the user. The crash reporter may be enabled by
// preference or through an environment variable, but the kDisableBreakpad
// switch overrides both.
« no previous file with comments | « components/breakpad/breakpad_client.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698