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

Unified Diff: chrome/browser/chrome_browser_main_win.cc

Issue 23513049: Implement install module verification metric. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Reviewer comments. Created 7 years, 3 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 16a64525c3436361ae119b913d4ea76e257f8623..7be7e239d467a122c7e43a5ac7097964026ca811 100644
--- a/chrome/browser/chrome_browser_main_win.cc
+++ b/chrome/browser/chrome_browser_main_win.cc
@@ -22,6 +22,7 @@
#include "base/win/windows_version.h"
#include "base/win/wrapped_window_proc.h"
#include "chrome/browser/browser_util_win.h"
+#include "chrome/browser/install_module_verifier_win.h"
#include "chrome/browser/profiles/profile_info_cache.h"
#include "chrome/browser/profiles/profile_shortcut_manager.h"
#include "chrome/browser/shell_integration.h"
@@ -38,6 +39,7 @@
#include "chrome/installer/util/install_util.h"
#include "chrome/installer/util/l10n_string_util.h"
#include "chrome/installer/util/shell_util.h"
+#include "content/public/browser/browser_thread.h"
#include "content/public/common/main_function_params.h"
#include "grit/app_locale_settings.h"
#include "grit/chromium_strings.h"
@@ -224,6 +226,18 @@ void ChromeBrowserMainPartsWin::ShowMissingLocaleMessageBox() {
MB_OK | MB_ICONERROR | MB_TOPMOST);
}
+void ChromeBrowserMainPartsWin::PostBrowserStart() {
+ ChromeBrowserMainParts::PostBrowserStart();
+
+ // Set up a task to verify installed modules in the current process. Use a
+ // delay to reduce the impact on startup time.
+ content::BrowserThread::GetMessageLoopProxyForThread(
+ content::BrowserThread::UI)->PostDelayedTask(
+ FROM_HERE,
+ base::Bind(&BeginModuleVerification),
+ base::TimeDelta::FromSeconds(45));
+}
+
// static
void ChromeBrowserMainPartsWin::PrepareRestartOnCrashEnviroment(
const CommandLine& parsed_command_line) {

Powered by Google App Engine
This is Rietveld 408576698