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

Side by Side Diff: chrome/app/chrome_crash_reporter_client.cc

Issue 819133004: Make callers of CommandLine use it via the base:: namespace. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix Created 5 years, 12 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 unified diff | Download patch
« no previous file with comments | « no previous file | chrome/app/chrome_crash_reporter_client_mac.mm » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/app/chrome_crash_reporter_client.h" 5 #include "chrome/app/chrome_crash_reporter_client.h"
6 6
7 #include "base/atomicops.h" 7 #include "base/atomicops.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/environment.h" 9 #include "base/environment.h"
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
(...skipping 322 matching lines...) Expand 10 before | Expand all | Expand 10 after
333 } 333 }
334 334
335 bool ChromeCrashReporterClient::GetCollectStatsConsent() { 335 bool ChromeCrashReporterClient::GetCollectStatsConsent() {
336 #if defined(GOOGLE_CHROME_BUILD) 336 #if defined(GOOGLE_CHROME_BUILD)
337 bool is_official_chrome_build = true; 337 bool is_official_chrome_build = true;
338 #else 338 #else
339 bool is_official_chrome_build = false; 339 bool is_official_chrome_build = false;
340 #endif 340 #endif
341 341
342 #if defined(OS_CHROMEOS) 342 #if defined(OS_CHROMEOS)
343 bool is_guest_session = CommandLine::ForCurrentProcess()->HasSwitch( 343 bool is_guest_session = base::CommandLine::ForCurrentProcess()->HasSwitch(
344 chromeos::switches::kGuestSession); 344 chromeos::switches::kGuestSession);
345 bool is_stable_channel = 345 bool is_stable_channel =
346 chrome::VersionInfo::GetChannel() == chrome::VersionInfo::CHANNEL_STABLE; 346 chrome::VersionInfo::GetChannel() == chrome::VersionInfo::CHANNEL_STABLE;
347 347
348 if (is_guest_session && is_stable_channel) 348 if (is_guest_session && is_stable_channel)
349 return false; 349 return false;
350 #endif // defined(OS_CHROMEOS) 350 #endif // defined(OS_CHROMEOS)
351 351
352 #if defined(OS_ANDROID) 352 #if defined(OS_ANDROID)
353 // TODO(jcivelli): we should not initialize the crash-reporter when it was not 353 // TODO(jcivelli): we should not initialize the crash-reporter when it was not
(...skipping 15 matching lines...) Expand all
369 bool ChromeCrashReporterClient::EnableBreakpadForProcess( 369 bool ChromeCrashReporterClient::EnableBreakpadForProcess(
370 const std::string& process_type) { 370 const std::string& process_type) {
371 return process_type == switches::kRendererProcess || 371 return process_type == switches::kRendererProcess ||
372 process_type == switches::kPluginProcess || 372 process_type == switches::kPluginProcess ||
373 process_type == switches::kPpapiPluginProcess || 373 process_type == switches::kPpapiPluginProcess ||
374 process_type == switches::kZygoteProcess || 374 process_type == switches::kZygoteProcess ||
375 process_type == switches::kGpuProcess; 375 process_type == switches::kGpuProcess;
376 } 376 }
377 377
378 } // namespace chrome 378 } // namespace chrome
OLDNEW
« no previous file with comments | « no previous file | chrome/app/chrome_crash_reporter_client_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698