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

Unified Diff: ash/common/system/chromeos/network/sms_observer.cc

Issue 2583393002: Send notification to users upon receiving sms messages (Closed)
Patch Set: update return type Created 3 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: ash/common/system/chromeos/network/sms_observer.cc
diff --git a/ash/common/system/chromeos/network/sms_observer.cc b/ash/common/system/chromeos/network/sms_observer.cc
new file mode 100644
index 0000000000000000000000000000000000000000..3b3894329a80c0313b3228d28f4529583d5f2176
--- /dev/null
+++ b/ash/common/system/chromeos/network/sms_observer.cc
@@ -0,0 +1,95 @@
+// Copyright 2017 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 <utility>
+
+#include "ash/common/material_design/material_design_controller.h"
+#include "ash/common/metrics/user_metrics_action.h"
+#include "ash/common/system/chromeos/network/sms_observer.h"
+#include "ash/common/system/system_notifier.h"
+#include "ash/common/system/tray/tray_constants.h"
+#include "ash/common/wm_shell.h"
+#include "ash/resources/vector_icons/vector_icons.h"
+#include "base/memory/ptr_util.h"
+#include "base/strings/string_number_conversions.h"
+#include "base/strings/utf_string_conversions.h"
+#include "chromeos/network/network_event_log.h"
+#include "chromeos/network/network_handler.h"
+#include "grit/ash_resources.h"
+#include "grit/ash_strings.h"
+#include "ui/base/l10n/l10n_util.h"
+#include "ui/base/resource/resource_bundle.h"
+#include "ui/gfx/paint_vector_icon.h"
+#include "ui/message_center/message_center.h"
+#include "ui/views/view.h"
+
+namespace ash {
+
+// Send the |message| to notification center to display to users. Note that each
+// notification will be assigned with different |message_id| as notification id.
+void SendMessage(const base::DictionaryValue* message,
+ std::string message_text,
+ std::string message_number,
+ int message_id) {
+ const char kNotificationId[] = "chrome://network/sms";
+ std::unique_ptr<message_center::Notification> notification;
+
+ notification.reset(new message_center::Notification(
+ message_center::NOTIFICATION_TYPE_SIMPLE,
+ kNotificationId + std::to_string(message_id),
+ base::ASCIIToUTF16(message_number), base::ASCIIToUTF16(message_text),
+ gfx::Image(
+ CreateVectorIcon(kSystemMenuSmsIcon, kMenuIconSize, kMenuIconColor)),
+ base::string16(), GURL(),
+ message_center::NotifierId(message_center::NotifierId::APPLICATION,
+ system_notifier::kNotifierSms),
+ message_center::RichNotificationData(), nullptr));
+ message_center::MessageCenter::Get()->AddNotification(
+ std::move(notification));
+}
+
+SmsObserver::SmsObserver() : message_id_(0) {
+ // TODO(armansito): SMS could be a special case for cellular that requires a
+ // user (perhaps the owner) to be logged in. If that is the case, then an
+ // additional check should be done before subscribing for SMS notifications.
+ if (chromeos::NetworkHandler::IsInitialized())
+ chromeos::NetworkHandler::Get()->network_sms_handler()->AddObserver(this);
+}
+
+SmsObserver::~SmsObserver() {
+ if (chromeos::NetworkHandler::IsInitialized()) {
+ chromeos::NetworkHandler::Get()->network_sms_handler()->RemoveObserver(
+ this);
+ }
+}
+
+void SmsObserver::MessageReceived(const base::DictionaryValue& message) {
+ std::string message_text;
+ if (!message.GetStringWithoutPathExpansion(
+ chromeos::NetworkSmsHandler::kTextKey, &message_text)) {
+ NET_LOG_ERROR("SMS message contains no content.", "");
+ return;
+ }
+ // TODO(armansito): A message might be due to a special "Message Waiting"
+ // state that the message is in. Once SMS handling moves to shill, such
+ // messages should be filtered there so that this check becomes unnecessary.
+ if (message_text.empty()) {
+ NET_LOG_DEBUG("SMS has empty content text. Ignoring.", "");
+ return;
+ }
+ std::string message_number;
+ if (!message.GetStringWithoutPathExpansion(
+ chromeos::NetworkSmsHandler::kNumberKey, &message_number)) {
+ NET_LOG_DEBUG("SMS contains no number. Ignoring.", "");
+ return;
+ }
+
+ NET_LOG_DEBUG(
+ "Received SMS from: " + message_number + " with text: " + message_text,
+ "");
+ message_id_++;
+ SendMessage(&message, message_text, message_number, message_id_);
+}
+
+} // namespace ash
« no previous file with comments | « ash/common/system/chromeos/network/sms_observer.h ('k') | ash/common/system/chromeos/network/sms_observer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698