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

Unified Diff: chromecast/crash/android/cast_crash_reporter_client_android.cc

Issue 1154383006: Adding crash utilities to chromecast/crash. (Closed) Base URL: https://eureka-internal.googlesource.com/chromium/src@master
Patch Set: cast_shell_unittests not built for android Created 5 years, 6 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
Index: chromecast/crash/android/cast_crash_reporter_client_android.cc
diff --git a/chromecast/crash/android/cast_crash_reporter_client_android.cc b/chromecast/crash/android/cast_crash_reporter_client_android.cc
deleted file mode 100644
index af5ec8dd843fd5e99d208bf8742f6d24659de153..0000000000000000000000000000000000000000
--- a/chromecast/crash/android/cast_crash_reporter_client_android.cc
+++ /dev/null
@@ -1,82 +0,0 @@
-// Copyright 2014 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 "chromecast/crash/android/cast_crash_reporter_client_android.h"
-
-#include "base/base_paths.h"
-#include "base/files/file_path.h"
-#include "base/files/file_util.h"
-#include "base/path_service.h"
-#include "chromecast/base/chromecast_config_android.h"
-#include "chromecast/base/version.h"
-#include "chromecast/common/global_descriptors.h"
-#include "chromecast/crash/cast_crash_keys.h"
-#include "content/public/common/content_switches.h"
-
-namespace chromecast {
-
-CastCrashReporterClientAndroid::CastCrashReporterClientAndroid(
- const std::string& process_type)
- : process_type_(process_type) {
-}
-
-CastCrashReporterClientAndroid::~CastCrashReporterClientAndroid() {
-}
-
-void CastCrashReporterClientAndroid::GetProductNameAndVersion(
- const char** product_name,
- const char** version) {
- *product_name = "media_shell";
- *version = PRODUCT_VERSION
-#if CAST_IS_DEBUG_BUILD()
- ".debug"
-#endif
- "." CAST_BUILD_REVISION;
-}
-
-base::FilePath CastCrashReporterClientAndroid::GetReporterLogFilename() {
- return base::FilePath(FILE_PATH_LITERAL("uploads.log"));
-}
-
-bool CastCrashReporterClientAndroid::GetCrashDumpLocation(
- base::FilePath* crash_dir) {
- base::FilePath crash_dir_local;
- if (!PathService::Get(base::DIR_ANDROID_APP_DATA, &crash_dir_local)) {
- return false;
- }
- crash_dir_local = crash_dir_local.Append("crashes");
-
- // Only try to create the directory in the browser process (empty value).
- if (process_type_.empty()) {
- if (!base::DirectoryExists(crash_dir_local)) {
- if (!base::CreateDirectory(crash_dir_local)) {
- return false;
- }
- }
- }
-
- // Provide value to crash_dir once directory is known to be a valid path.
- *crash_dir = crash_dir_local;
- return true;
-}
-
-size_t CastCrashReporterClientAndroid::RegisterCrashKeys() {
- return crash_keys::RegisterCastCrashKeys();
-}
-
-bool CastCrashReporterClientAndroid::GetCollectStatsConsent() {
- return android::ChromecastConfigAndroid::GetInstance()->CanSendUsageStats();
-}
-
-int CastCrashReporterClientAndroid::GetAndroidMinidumpDescriptor() {
- return kAndroidMinidumpDescriptor;
-}
-
-bool CastCrashReporterClientAndroid::EnableBreakpadForProcess(
- const std::string& process_type) {
- return process_type == switches::kRendererProcess ||
- process_type == switches::kGpuProcess;
-}
-
-} // namespace chromecast
« no previous file with comments | « chromecast/crash/android/cast_crash_reporter_client_android.h ('k') | chromecast/crash/android/crash_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698