[Mac] Add XPC alerts to GetDisplayedNotifications

chrome/browser/notifications/stub_alert_dispatcher_mac.h

Index: chrome/browser/notifications/stub_alert_dispatcher_mac.h
diff --git a/chrome/browser/notifications/stub_alert_dispatcher_mac.h b/chrome/browser/notifications/stub_alert_dispatcher_mac.h
index 29af5a3fc4ef8bbc348247f89bcd8e96d437d69c..ec75959dcc382cf8b112f3354c8ee1fb61e4ffa2 100644
--- a/chrome/browser/notifications/stub_alert_dispatcher_mac.h
+++ b/chrome/browser/notifications/stub_alert_dispatcher_mac.h
@@ -7,7 +7,17 @@
 
 #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>
 
@@ -18,6 +28,11 @@
 
 - (void)closeAllNotifications;
 
+- (void)getDisplayedAlertsForProfileId:(NSString*)profileId
+                withNotificationCenter:
+                    (NSUserNotificationCenter*)notificationCenter
+                              callback:(DisplayedNotificationsCallback)callback;
+
 // Stub specific methods.
 - (NSArray*)alerts;