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

Unified Diff: chrome/browser/chrome_browser_main_win.cc

Issue 2621363002: Remove Kasko! (Closed)
Patch Set: 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
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 1c782dad8e77e91b9ff7e0b79a8c90e1414fc22e..9b7c29d0c666f324160ad635d32ec2ba5b194f67 100644
--- a/chrome/browser/chrome_browser_main_win.cc
+++ b/chrome/browser/chrome_browser_main_win.cc
@@ -71,10 +71,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)(
@@ -108,41 +104,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,
@@ -340,18 +301,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(

Powered by Google App Engine
This is Rietveld 408576698