[Mac] Add XPC alerts to GetDisplayedNotifications

chrome/browser/notifications/notification_platform_bridge_android.cc

 // Copyright 2014 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 "chrome/browser/notifications/notification_platform_bridge_android.h"
 
 #include <utility>
 #include <vector>
 
 #include "base/android/jni_array.h"
@@ skipping 301 matching lines @@
 }
 
 bool NotificationPlatformBridgeAndroid::GetDisplayed(
     const std::string& profile_id,
     bool incognito,
     std::set<std::string>* notifications) const {
   // TODO(miguelg): This can actually be implemented for M+
   return false;
 }
 
+void NotificationPlatformBridgeMac::GetDisplayedAsync(
+    const std::string& profile_id,
+    bool incognito,
+    const NotificationCommon::NotificationResultCallback& callback) const {
+  std::unique_ptr<std::set<std::string>> displayedNotifications =
+      base::MakeUnique<std::set<std::string>>();
+  content::BrowserThread::PostTask(
+      content::BrowserThread::IO, FROM_HERE,
+      base::Bind(callback, base::Passed(&displayedNotifications),
+                 false /* supports sync */));

[Peter Beverloo, 2017/02/24 16:12:24]

nit: "sync" is rather ambiguous (also in the other impl) when the method name
includes "async"

There's plenty of space here, what about:

  false /* supports getting displayed notifications */

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

Done.

+}
+
 // static
 bool NotificationPlatformBridgeAndroid::RegisterNotificationPlatformBridge(
     JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 // static
 void NotificationPlatformBridgeAndroid::RegisterProfilePrefs(
     user_prefs::PrefRegistrySyncable* registry) {
   registry->RegisterBooleanPref(prefs::kNotificationsVibrateEnabled, true);
 }
 
 NotificationPlatformBridgeAndroid::RegeneratedNotificationInfo::
     RegeneratedNotificationInfo() {}
 
 NotificationPlatformBridgeAndroid::RegeneratedNotificationInfo::
     RegeneratedNotificationInfo(const std::string& origin,
                                 const std::string& tag,
                                 const std::string& webapk_package)
     : origin(origin), tag(tag), webapk_package(webapk_package) {}
 
 NotificationPlatformBridgeAndroid::RegeneratedNotificationInfo::
     ~RegeneratedNotificationInfo() {}