Support buttons on ARC Notifications

ui/arc/notification/arc_notification_manager.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_manager.h"
 
 #include "base/stl_util.h"
 #include "ui/arc/notification/arc_notification_item.h"
 
 namespace arc {
@@ skipping 73 matching lines @@
   if (!items_.contains(key)) {
     VLOG(3) << "Chrome requests to fire a click event on notification (key: "
             << key << "), but it is gone.";
     return;
   }
 
   arc_bridge_->notifications_instance()->SendNotificationEventToAndroid(
       key, ARC_NOTIFICATION_EVENT_BODY_CLICKED);
 }
 
+void ArcNotificationManager::SendNotificationButtonClickedOnChrome(
+    const std::string& key, int button_index) {
+  if (!items_.contains(key)) {
+    VLOG(3) << "Chrome requests to fire a click event on notification (key: "
+            << key << "), but it is gone.";

[hidehiko, 2016/01/22 07:47:03]

nit: maybe "it has gone"?

[yoshiki, 2016/01/25 15:45:50]

Done.

+    return;
+  }
+
+  arc::ArcNotificationEvent command;
+  switch (button_index) {
+    case 0:  command = ARC_NOTIFICATION_EVENT_BUTTON1_CLICKED;  break;

[hidehiko, 2016/01/22 07:47:03]

Let's follow the style guide:
https://google.github.io/styleguide/cppguide.html#Loops_and_Switch_Statements

[yoshiki, 2016/01/25 15:45:50]

Done.

+    case 1:  command = ARC_NOTIFICATION_EVENT_BUTTON2_CLICKED;  break;
+    case 2:  command = ARC_NOTIFICATION_EVENT_BUTTON3_CLICKED;  break;
+    case 3:  command = ARC_NOTIFICATION_EVENT_BUTTON4_CLICKED;  break;
+    case 4:  command = ARC_NOTIFICATION_EVENT_BUTTON5_CLICKED;  break;
+    default:
+      VLOG(3) << "Invalid button index (key: " << key << ", index: " <<
+          button_index << ").";
+      return;
+  }
+
+  arc_bridge_->notifications_instance()->SendNotificationEventToAndroid(
+      key, command);
+}
+
 }  // namespace arc