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

Unified Diff: chrome/browser/first_run/first_run_win.cc

Issue 7377010: This change will split the result codes between content and chrome. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: actually rename the files Created 9 years, 5 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/first_run/first_run_win.cc
diff --git a/chrome/browser/first_run/first_run_win.cc b/chrome/browser/first_run/first_run_win.cc
index a5b1e91e33f71574c3042d1686cb796f57b40779..973f5c945b94575c07bd6612eebf285c4aee290e 100644
--- a/chrome/browser/first_run/first_run_win.cc
+++ b/chrome/browser/first_run/first_run_win.cc
@@ -28,6 +28,7 @@
#include "chrome/browser/importer/importer_progress_dialog.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/common/chrome_notification_types.h"
+#include "chrome/common/chrome_result_codes.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/worker_thread_ticker.h"
#include "chrome/installer/util/browser_distribution.h"
@@ -38,7 +39,6 @@
#include "chrome/installer/util/util_constants.h"
#include "content/browser/user_metrics.h"
#include "content/common/notification_service.h"
-#include "content/common/result_codes.h"
#include "google_update_idl.h"
#include "grit/chromium_strings.h"
#include "grit/generated_resources.h"
@@ -183,7 +183,7 @@ class ImportProcessRunner : public base::win::ObjectWatcher::Delegate {
// the import_process handle.
explicit ImportProcessRunner(base::ProcessHandle import_process)
: import_process_(import_process),
- exit_code_(ResultCodes::NORMAL_EXIT) {
+ exit_code_(content::RESULT_CODE_NORMAL_EXIT) {
watcher_.StartWatching(import_process, this);
MessageLoop::current()->Run();
}
@@ -245,7 +245,7 @@ class HungImporterMonitor : public WorkerThreadTicker::Callback {
// while the other process still not pumping messages.
HWND active_window = ::GetLastActivePopup(owner_window_);
if (::IsHungAppWindow(active_window) || ::IsHungAppWindow(owner_window_)) {
- ::TerminateProcess(import_process_, ResultCodes::IMPORTER_HUNG);
+ ::TerminateProcess(import_process_, chrome::RESULT_CODE_IMPORTER_HUNG);
import_process_ = NULL;
}
}
@@ -355,7 +355,7 @@ bool FirstRun::ImportSettings(Profile* profile,
if (profile)
profile->GetPrefs()->ReloadPersistentPrefs();
- return (import_runner.exit_code() == ResultCodes::NORMAL_EXIT);
+ return (import_runner.exit_code() == content::RESULT_CODE_NORMAL_EXIT);
}
// static

Powered by Google App Engine
This is Rietveld 408576698