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

Unified Diff: chrome/browser/chromeos/login/version_info_updater.cc

Issue 22262004: Drop struct BootStat and all usage (boot stats are now sent from chromeos code). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 4 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/chromeos/login/version_info_updater.cc
diff --git a/chrome/browser/chromeos/login/version_info_updater.cc b/chrome/browser/chromeos/login/version_info_updater.cc
index 5d430c7b4c70ed3501cc47d3d5660dd321b65d71..84ff2ce3dba8f6e4ef39829ce28d6d30e1644118 100644
--- a/chrome/browser/chromeos/login/version_info_updater.cc
+++ b/chrome/browser/chromeos/login/version_info_updater.cc
@@ -66,11 +66,6 @@ void VersionInfoUpdater::StartUpdate(bool is_official_build) {
base::Bind(&VersionInfoUpdater::OnVersion,
weak_pointer_factory_.GetWeakPtr()),
&tracker_);
- boot_times_loader_.GetBootTimes(
- base::Bind(is_official_build ? &VersionInfoUpdater::OnBootTimesNoop
- : &VersionInfoUpdater::OnBootTimes,
- weak_pointer_factory_.GetWeakPtr()),
- &tracker_);
} else {
UpdateVersionLabel();
}
@@ -131,39 +126,6 @@ void VersionInfoUpdater::OnVersion(const std::string& version) {
UpdateVersionLabel();
}
-void VersionInfoUpdater::OnBootTimesNoop(
- const BootTimesLoader::BootTimes& boot_times) {}
-
-void VersionInfoUpdater::OnBootTimes(
- const BootTimesLoader::BootTimes& boot_times) {
- const char* kBootTimesNoChromeExec =
- "Non-firmware boot took %.2f seconds (kernel %.2fs, system %.2fs)";
- const char* kBootTimesChromeExec =
- "Non-firmware boot took %.2f seconds "
- "(kernel %.2fs, system %.2fs, chrome %.2fs)";
- std::string boot_times_text;
-
- if (boot_times.chrome > 0) {
- boot_times_text =
- base::StringPrintf(
- kBootTimesChromeExec,
- boot_times.total,
- boot_times.pre_startup,
- boot_times.system,
- boot_times.chrome);
- } else {
- boot_times_text =
- base::StringPrintf(
- kBootTimesNoChromeExec,
- boot_times.total,
- boot_times.pre_startup,
- boot_times.system);
- }
- // Use UTF8ToWide once this string is localized.
- if (delegate_)
- delegate_->OnBootTimesLabelTextUpdated(boot_times_text);
-}
-
void VersionInfoUpdater::OnStoreLoaded(policy::CloudPolicyStore* store) {
UpdateEnterpriseInfo();
}
« no previous file with comments | « chrome/browser/chromeos/login/version_info_updater.h ('k') | chrome/browser/resources/chromeos/login/version.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698