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

Unified Diff: chrome/browser/chromeos/ui/low_disk_notification.cc

Issue 2082363004: Show notifications on low disk space. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@low-disk-strings
Patch Set: Created 4 years, 6 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/ui/low_disk_notification.cc
diff --git a/chrome/browser/chromeos/ui/low_disk_notification.cc b/chrome/browser/chromeos/ui/low_disk_notification.cc
new file mode 100644
index 0000000000000000000000000000000000000000..974fcd534960e144b7ea159ef9c8d462b277b486
--- /dev/null
+++ b/chrome/browser/chromeos/ui/low_disk_notification.cc
@@ -0,0 +1,101 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/chromeos/ui/low_disk_notification.h"
+
+#include <stdint.h>
+
+#include "ash/common/system/system_notifier.h"
+#include "base/bind.h"
+#include "base/macros.h"
+#include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/grit/generated_resources.h"
+#include "chromeos/dbus/cryptohome_client.h"
+#include "ui/base/l10n/l10n_util.h"
+#include "ui/base/resource/resource_bundle.h"
+#include "ui/chromeos/resources/grit/ui_chromeos_resources.h"
+#include "ui/message_center/message_center.h"
+#include "ui/message_center/notification.h"
+#include "ui/message_center/notification_types.h"
+#include "ui/message_center/notifier_settings.h"
+
+namespace {
+const char kLowDiskId[] = "low_disk";
+const uint64_t kNotificationThreshold = 1 << 30; // 1GB
+const uint64_t kNotificationSevereThreshold = 512 << 20; // 512MB
+const base::TimeDelta kNotificationInterval = base::TimeDelta::FromMinutes(2);
yoshiki 2016/06/23 08:00:45 constexpr is better than const.
dspaid 2016/06/27 05:40:04 Done.
+} // namespace
+
+namespace chromeos {
+
+LowDiskNotification::LowDiskNotification(CryptohomeClient* cryptohome_client)
+ : cryptohome_client_(cryptohome_client) {
yoshiki 2016/06/23 08:00:45 I think it's better to use DBusThreadManager::Get(
dspaid 2016/06/27 05:40:04 Done.
+ if (cryptohome_client_) {
yoshiki 2016/06/23 08:00:45 cryptohome_client_ is necessary for this class, so
dspaid 2016/06/27 05:40:04 Done.
+ cryptohome_client_->SetLowDiskSpaceHandler(base::Bind(
+ &LowDiskNotification::OnLowDiskSpace, base::Unretained(this)));
yoshiki 2016/06/23 08:00:45 You might need to close the notification when the
dspaid 2016/06/27 05:40:04 We don't actually have this information. Notifica
+ }
+}
+
+LowDiskNotification::~LowDiskNotification() {
yoshiki 2016/06/23 08:00:45 Please DCHECK(DBusThreadManager::Get()->GetCryptoh
dspaid 2016/06/27 05:40:04 Done.
+ if (cryptohome_client_)
+ cryptohome_client_->ResetLowDiskSpaceHandler();
+}
+
+void LowDiskNotification::OnLowDiskSpace(uint64_t free_disk_bytes) {
+ LOG(ERROR) << "LowDiskNotification received";
+ Severity severity = GetSeverity(free_disk_bytes);
+ base::Time now = base::Time::Now();
+ if (severity > last_notification_severity_ ||
yoshiki 2016/06/23 08:00:45 Shouldn't we update the notification content when
dspaid 2016/06/27 05:40:04 Done.
+ now - last_notification_time_ > kNotificationInterval) {
+ g_browser_process->message_center()->AddNotification(
yoshiki 2016/06/23 08:00:45 You shouldn't pass nullptr to AddNotification.
dspaid 2016/06/27 05:40:04 Done.
+ CreateNotification(severity));
+ last_notification_time_ = now;
+ last_notification_severity_ = severity;
+ }
+}
+
+std::unique_ptr<message_center::Notification>
+LowDiskNotification::CreateNotification(Severity severity) {
+ std::unique_ptr<message_center::Notification> notification;
+ switch (severity) {
+ case MEDIUM:
+ notification = message_center::Notification::CreateSystemNotification(
+ kLowDiskId,
+ l10n_util::GetStringUTF16(IDS_LOW_DISK_NOTIFICATION_TITLE),
yoshiki 2016/06/23 08:00:45 Did you forget to add the string grd file to this
dspaid 2016/06/27 05:40:04 They were in a separate change which has now been
+ l10n_util::GetStringUTF16(IDS_LOW_DISK_NOTIFICATION_MESSAGE),
+ gfx::Image(ui::ResourceBundle::GetSharedInstance().GetImageNamed(
+ IDR_DISK_SPACE_NOTIFICATION_LOW)),
+ ash::system_notifier::kNotifierDisk, base::Closure());
+ break;
+ case HIGH:
+ notification = message_center::Notification::CreateSystemNotification(
yoshiki 2016/06/23 08:00:45 Parhaps, pinned notification is better? https://cs
dspaid 2016/06/27 05:40:04 Done.
+ kLowDiskId,
+ l10n_util::GetStringUTF16(IDS_CRITICALLY_LOW_DISK_NOTIFICATION_TITLE),
+ l10n_util::GetStringUTF16(
+ IDS_CRITICALLY_LOW_DISK_NOTIFICATION_MESSAGE),
+ gfx::Image(ui::ResourceBundle::GetSharedInstance().GetImageNamed(
+ IDR_DISK_SPACE_NOTIFICATION_CRITICAL)),
+ ash::system_notifier::kNotifierDisk, base::Closure());
+ break;
+ default:
+ return nullptr;
+ }
+ std::vector<message_center::ButtonInfo> buttons;
+ buttons.push_back(message_center::ButtonInfo(
+ l10n_util::GetStringUTF16(IDS_LOW_DISK_NOTIFICATION_BUTTON)));
yoshiki 2016/06/23 08:00:45 Don't you add a click handler for this button?
dspaid 2016/06/27 05:40:04 Done.
+ notification->set_buttons(buttons);
+ return notification;
+}
+
+LowDiskNotification::Severity LowDiskNotification::GetSeverity(
+ uint64_t free_disk_bytes) {
+ if (free_disk_bytes < kNotificationSevereThreshold)
+ return Severity::HIGH;
+ if (free_disk_bytes < kNotificationThreshold)
+ return Severity::MEDIUM;
+ return Severity::NONE;
+}
+
+} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698