ARC: Support Android Notification Priority

ui/arc/notification/arc_notification_item.cc

 // 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 "ui/arc/notification/arc_notification_item.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/strings/string16.h"
@@ skipping 21 matching lines @@
   DCHECK(base::WorkerPool::RunsTasksOnCurrentThread());
   DCHECK(!data.empty());  // empty string should be handled in caller.
 
   // We may decode an image in the browser process since it has been generated
   // in NotificationListerService in Android and should be safe.
   SkBitmap bitmap;
   gfx::PNGCodec::Decode(&data[0], data.size(), &bitmap);
   return bitmap;
 }
 
+int convertAndroidPriority(const int androidPriority) {

[hidehiko, 2016/04/28 17:07:48]

Could you comment your intention of this mapping?

[yoshiki, 2016/04/28 18:09:07]

Done.

+  switch (androidPriority) {
+    case -2:  // PRIORITY_MIN
+    case -1:  // PRIORITY_LOW
+      return -2;
+    case 0:   // PRIORITY_DEFAULT
+      return -1;
+    case 1:   // PRIORITY_HIGH
+      return 0;
+    case 2:   // PRIORITY_MAX
+      return 2;
+    default:
+      NOTREACHED() << "Invalid Priority";
+      return 0;

[hidehiko, 2016/04/28 17:07:48]

Maybe -1 not to show the popup?

[yoshiki, 2016/04/28 18:09:07]

I think the default priority (= 0) should be used since this is invalid android
priority.

+  }
+}
+
 class ArcNotificationDelegate : public message_center::NotificationDelegate {
  public:
   explicit ArcNotificationDelegate(base::WeakPtr<ArcNotificationItem> item)
       : item_(item) {}
 
   void Close(bool by_user) override {
     if (item_)
       item_->Close(by_user);
   }
 
@@ skipping 80 matching lines @@
 
   for (size_t i = 0; i < data.buttons.size(); i++) {
     rich_data.buttons.push_back(message_center::ButtonInfo(
         base::UTF8ToUTF16(data.buttons.at(i)->label.get())));
   }
 
   // If the client is old (version < 1), both |no_clear| and |ongoing_event|
   // are false.
   rich_data.pinned = (data.no_clear || data.ongoing_event);
 
+  rich_data.priority = convertAndroidPriority(data.priority);
+
   // The identifier of the notifier, which is used to distinguish the notifiers
   // in the message center.
   message_center::NotifierId notifier_id(
       message_center::NotifierId::SYSTEM_COMPONENT, kNotifierId);
   notifier_id.profile_id = profile_id_.GetUserEmail();
 
   DCHECK(!data.title.is_null());
   DCHECK(!data.message.is_null());
   notification_.reset(new message_center::Notification(
       type, notification_id_, base::UTF8ToUTF16(data.title.get()),
@@ skipping 56 matching lines @@
   message_center_->AddNotification(std::move(notification_));
 
   if (newer_data_) {
     // There is the newer data, so updates again.
     mojom::ArcNotificationDataPtr data(std::move(newer_data_));
     UpdateWithArcNotificationData(*data);
   }
 }
 
 }  // namespace arc