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

Unified Diff: chrome/browser/notifications/native_notification_display_service.cc

Issue 2418153002: Hooking up Android N inline notification replies end-to-end (Closed)
Patch Set: Reformat getNotificationReply Created 4 years, 2 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/notifications/native_notification_display_service.cc
diff --git a/chrome/browser/notifications/native_notification_display_service.cc b/chrome/browser/notifications/native_notification_display_service.cc
index d7271b0340ef3052568e604106823a18642ed405..2de879d0bee177fe377d7bfd028db38317b7c310 100644
--- a/chrome/browser/notifications/native_notification_display_service.cc
+++ b/chrome/browser/notifications/native_notification_display_service.cc
@@ -80,13 +80,13 @@ void NativeNotificationDisplayService::ProcessNotificationOperation(
NotificationCommon::Type notification_type,
const std::string& origin,
const std::string& notification_id,
- int action_index) {
+ int action_index,
+ const base::NullableString16& reply) {
NotificationHandler* handler = GetNotificationHandler(notification_type);
CHECK(handler);
switch (operation) {
case NotificationCommon::CLICK:
- handler->OnClick(profile_, origin, notification_id, action_index,
- base::NullableString16() /* reply */);
+ handler->OnClick(profile_, origin, notification_id, action_index, reply);
break;
case NotificationCommon::CLOSE:
handler->OnClose(profile_, origin, notification_id, true /* by_user */);

Powered by Google App Engine
This is Rietveld 408576698