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

Unified Diff: chrome/browser/chrome_browser_main_win.cc

Issue 2621363002: Remove Kasko! (Closed)
Patch Set: Rebase Created 3 years, 11 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 | « chrome/browser/chrome_browser_main_win.h ('k') | chrome/chrome.isolate » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chrome_browser_main_win.cc
diff --git a/chrome/browser/chrome_browser_main_win.cc b/chrome/browser/chrome_browser_main_win.cc
index dd3a9b6503670b9e48fde706f17daab7b46525fd..1e0113dc688d765c1e13b95140624bb4cb2129fa 100644
--- a/chrome/browser/chrome_browser_main_win.cc
+++ b/chrome/browser/chrome_browser_main_win.cc
@@ -77,10 +77,6 @@
#include "chrome/browser/google/did_run_updater_win.h"
#endif
-#if BUILDFLAG(ENABLE_KASKO)
-#include "syzygy/kasko/api/reporter.h"
-#endif
-
namespace {
typedef HRESULT (STDAPICALLTYPE* RegisterApplicationRestartProc)(
@@ -114,41 +110,6 @@ class TranslationDelegate : public installer::TranslationDelegate {
base::string16 GetLocalizedString(int installer_string_id) override;
};
-#if BUILDFLAG(ENABLE_KASKO)
-void ObserveFailedCrashReportDirectory(const base::FilePath& path, bool error) {
- DCHECK(!error);
- if (error)
- return;
- base::FileEnumerator enumerator(path, true, base::FileEnumerator::FILES);
- for (base::FilePath report_file = enumerator.Next(); !report_file.empty();
- report_file = enumerator.Next()) {
- if (report_file.Extension() ==
- kasko::api::kPermanentFailureMinidumpExtension) {
- UMA_HISTOGRAM_BOOLEAN("CrashReport.PermanentUploadFailure", true);
- }
- bool result = base::DeleteFile(report_file, false);
- DCHECK(result);
- }
-}
-
-void StartFailedKaskoCrashReportWatcher(base::FilePathWatcher* watcher) {
- base::FilePath watcher_data_directory;
- if (!PathService::Get(chrome::DIR_WATCHER_DATA, &watcher_data_directory)) {
- NOTREACHED();
- } else {
- base::FilePath permanent_failure_directory =
- watcher_data_directory.Append(kPermanentlyFailedReportsSubdir);
- if (!watcher->Watch(permanent_failure_directory, true,
- base::Bind(&ObserveFailedCrashReportDirectory))) {
- NOTREACHED();
- }
-
- // Call it once to observe any files present prior to the Watch() call.
- ObserveFailedCrashReportDirectory(permanent_failure_directory, false);
- }
-}
-#endif // BUILDFLAG(ENABLE_KASKO)
-
void DetectFaultTolerantHeap() {
enum FTHFlags {
FTH_HKLM = 1,
@@ -406,18 +367,6 @@ void ChromeBrowserMainPartsWin::PostBrowserStart() {
InitializeChromeElf();
-#if BUILDFLAG(ENABLE_KASKO)
- content::BrowserThread::PostDelayedTask(
- content::BrowserThread::FILE, FROM_HERE,
- base::Bind(&StartFailedKaskoCrashReportWatcher,
- base::Unretained(&failed_kasko_crash_report_watcher_)),
- base::TimeDelta::FromMinutes(5));
-#endif // BUILDFLAG(ENABLE_KASKO)
-
-#if defined(GOOGLE_CHROME_BUILD)
- did_run_updater_.reset(new DidRunUpdater);
-#endif
-
// Record UMA data about whether the fault-tolerant heap is enabled.
// Use a delayed task to minimize the impact on startup time.
content::BrowserThread::PostDelayedTask(
« no previous file with comments | « chrome/browser/chrome_browser_main_win.h ('k') | chrome/chrome.isolate » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698