[Mac] Add XPC alerts to GetDisplayedNotifications

chrome/browser/notifications/stub_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_STUB_ALERT_DISPATCHER_MAC_H_
 #define CHROME_BROWSER_NOTIFICATIONS_STUB_ALERT_DISPATCHER_MAC_H_
 
 #import <Foundation/Foundation.h>
 
-#import "chrome/browser/notifications/alert_dispatcher_mac.h"
+#include <set>
+#include <string>
+
+#include "chrome/browser/notifications/alert_dispatcher_mac.h"
+
+namespace {
+
+using DisplayedNotificationsCallback =

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

Why is this re-declared instead of using the one in alert_dispatcher_mac.h?

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

Moved to inlining as per Peter's comment about anonymous namespaces in .h files
(but yes, in the original patch it could just use that).

+    base::Callback<void(std::unique_ptr<std::set<std::string>>,
+                        bool /* supports_synchronization */)>;
+}  // namespace

[Peter Beverloo, 2017/04/03 12:33:29]

dito re: anonymous namespaces in header files

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

Acknowledged.

 
 @interface StubAlertDispatcher : NSObject<AlertDispatcher>
 
 - (void)dispatchNotification:(NSDictionary*)data;

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

Similarly, you can just remove the AlertDispatcher methods here, too.

 
 - (void)closeNotificationWithId:(NSString*)notificationId
                   withProfileId:(NSString*)profileId;
 
 - (void)closeAllNotifications;
 
+- (void)getDisplayedAlertsForProfileId:(NSString*)profileId
+                withNotificationCenter:
+                    (NSUserNotificationCenter*)notificationCenter
+                              callback:(DisplayedNotificationsCallback)callback;
+
 // Stub specific methods.
 - (NSArray*)alerts;
 
 @end
 
 #endif  // CHROME_BROWSER_NOTIFICATIONS_STUB_ALERT_DISPATCHER_MAC_H_