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

Unified Diff: chrome/browser/chromeos/extensions/file_manager/device_event_router.cc

Issue 888083004: Revert "Files.app: Add UMA for HardUnplugged events." (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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/extensions/file_manager/device_event_router.cc
diff --git a/chrome/browser/chromeos/extensions/file_manager/device_event_router.cc b/chrome/browser/chromeos/extensions/file_manager/device_event_router.cc
index 3de49b32baf525ea1ebfae0b1fbbfd4f66ab0cdf..e269073c698723d1609fa713004095435ad160ef 100644
--- a/chrome/browser/chromeos/extensions/file_manager/device_event_router.cc
+++ b/chrome/browser/chromeos/extensions/file_manager/device_event_router.cc
@@ -3,7 +3,6 @@
// found in the LICENSE file.
#include "base/bind.h"
-#include "base/metrics/histogram_macros.h"
#include "base/thread_task_runner_handle.h"
#include "chrome/browser/chromeos/extensions/file_manager/device_event_router.h"
#include "chrome/browser/chromeos/file_manager/volume_manager.h"
@@ -81,13 +80,6 @@ void DeviceEventRouter::OnDiskRemoved(
const std::string& device_path = disk.system_path_prefix();
if (!disk.mount_path().empty() &&
GetDeviceState(device_path) != DEVICE_HARD_UNPLUGGED_AND_REPORTED) {
- // TODO(hirono): Remove the temporary UMA. crbug.com/433734
- if (!last_suspend_done_.is_null()) {
- UMA_HISTOGRAM_MEDIUM_TIMES(
- "FileBrowser.HardUnpluggedAroundSuspend.TimeSinceResume",
- base::Time::Now() - last_suspend_done_);
- }
- last_hard_unplugged_ = base::Time::Now();
OnDeviceEvent(file_manager_private::DEVICE_EVENT_TYPE_HARD_UNPLUGGED,
device_path);
SetDeviceState(device_path, DEVICE_HARD_UNPLUGGED_AND_REPORTED);
@@ -132,18 +124,11 @@ void DeviceEventRouter::OnFormatCompleted(const std::string& device_path,
void DeviceEventRouter::SuspendImminent() {
DCHECK(thread_checker_.CalledOnValidThread());
- // TODO(hirono): Remove the temporary UMA. crbug.com/433734
- if (!last_hard_unplugged_.is_null()) {
- UMA_HISTOGRAM_MEDIUM_TIMES(
- "FileBrowser.HardUnpluggedAroundSuspend.TimeUntilSuspend",
- base::Time::Now() - last_hard_unplugged_);
- }
is_resuming_ = true;
}
void DeviceEventRouter::SuspendDone(const base::TimeDelta& sleep_duration) {
DCHECK(thread_checker_.CalledOnValidThread());
- last_suspend_done_ = base::Time::Now();
base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
FROM_HERE,
base::Bind(&DeviceEventRouter::SuspendDoneDelayed,
« no previous file with comments | « chrome/browser/chromeos/extensions/file_manager/device_event_router.h ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698