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

Unified Diff: chrome/browser/gtk/first_run_dialog.cc

Issue 209042: Make Linux Breakpad initialization call to lsb_release asynchronous. Also wrap more... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 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/gtk/first_run_dialog.cc
===================================================================
--- chrome/browser/gtk/first_run_dialog.cc (revision 26606)
+++ chrome/browser/gtk/first_run_dialog.cc (working copy)
@@ -7,7 +7,6 @@
#include "app/l10n_util.h"
#include "app/resource_bundle.h"
#include "base/message_loop.h"
-#include "chrome/app/breakpad_linux.h"
#include "chrome/browser/gtk/gtk_chrome_link_button.h"
#include "chrome/browser/shell_integration.h"
#include "chrome/common/gtk_util.h"
@@ -17,6 +16,10 @@
#include "grit/google_chrome_strings.h"
#include "grit/locale_settings.h"
+#if defined(USE_LINUX_BREAKPAD)
+#include "chrome/app/breakpad_linux.h"
+#endif
+
// static
bool FirstRunDialog::Show(Profile* profile) {
int response = -1;
@@ -134,9 +137,11 @@
// Check if user has opted into reporting.
if (report_crashes_ &&
gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(report_crashes_))) {
+#if defined(USE_LINUX_BREAKPAD)
if (GoogleUpdateSettings::SetCollectStatsConsent(true)) {
InitCrashReporter();
}
+#endif
} else {
GoogleUpdateSettings::SetCollectStatsConsent(false);
}

Powered by Google App Engine
This is Rietveld 408576698