[Mac] Add XPC alerts to GetDisplayedNotifications

chrome/browser/notifications/alert_dispatcher_mac.h

 // 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.
 
 #ifndef CHROME_BROWSER_NOTIFICATIONS_ALERT_DISPATCHER_MAC_H_
 #define CHROME_BROWSER_NOTIFICATIONS_ALERT_DISPATCHER_MAC_H_
 
 #import <Foundation/Foundation.h>
 
+#include "chrome/browser/notifications/notification_common.h"
+
 // Interface to communicate with the Alert XPC service.
 @protocol AlertDispatcher<NSObject>
 
 // Deliver a notification to the XPC service to be displayed as an alert.
 - (void)dispatchNotification:(NSDictionary*)data;
 
 // Close a notification for a given |notificationId| and |profileId|.
 - (void)closeNotificationWithId:(NSString*)notificationId
                   withProfileId:(NSString*)profileId;
 
 // Close all notifications.
 - (void)closeAllNotifications;
 
+// Get currently displayed notifications.
+- (void)getDisplayedAlertIds:(NSString*)profileId

[Robert Sesek, 2017/02/24 22:38:05]

Maybe getDisplayedAlertsForProfileId:withLocalNotifications:callback: instead
(Cocoa naming is more art than science…).

[Miguel Garcia, 2017/03/22 22:00:06]

Done, but shouldn't it be withCallback or andCallback ? Trying to learn the art
:)

[Robert Sesek, 2017/04/03 16:51:24]

Methods with a lot of parameters get too wordy if every one is described as
"with" so it can be omitted after the first. It should read like a sentence, and
so you can think of the many parameters as being in a comma-separated serial
list (omitting the final conjunction).

[Miguel Garcia, 2017/04/04 12:21:26]

Acknowledged.

+      withLocalNotifications:(NSArray*)localNotifications
+                withCallback:
+                    (NotificationCommon::NotificationResultCallback)callback;
 @end
 
 #endif  // CHROME_BROWSER_NOTIFICATIONS_ALERT_DISPATCHER_MAC_H_