Make GetDisplayedNotifications asynchronous.

chrome/browser/notifications/notification_platform_bridge_android.cc

Index: chrome/browser/notifications/notification_platform_bridge_android.cc
diff --git a/chrome/browser/notifications/notification_platform_bridge_android.cc b/chrome/browser/notifications/notification_platform_bridge_android.cc
index 0c8c82fe6f819c043ad8bc8419020a02332cd819..2b16bc6ea6ca06d31aa363a5d158122968690c43 100644
--- a/chrome/browser/notifications/notification_platform_bridge_android.cc
+++ b/chrome/browser/notifications/notification_platform_bridge_android.cc
@@ -25,6 +25,7 @@
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "components/pref_registry/pref_registry_syncable.h"
+#include "content/public/browser/browser_thread.h"
 #include "content/public/common/persistent_notification_status.h"
 #include "content/public/common/platform_notification_data.h"
 #include "jni/ActionInfo_jni.h"
@@ -311,12 +312,16 @@ void NotificationPlatformBridgeAndroid::Close(
       webapk_package);
 }
 
-bool NotificationPlatformBridgeAndroid::GetDisplayed(
+void 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;
+    const NotificationResultCallback& callback) const {
+  std::unique_ptr<std::set<std::string>> displayedNotifications =

[Peter Beverloo, 2017/03/15 18:07:50]

nit: displayed_notifications

[Miguel Garcia, 2017/03/16 14:57:42]

Done.

+      base::MakeUnique<std::set<std::string>>();
+  content::BrowserThread::PostTask(
+      content::BrowserThread::UI, FROM_HERE,
+      base::Bind(callback, base::Passed(&displayedNotifications),
+                 false /* supports getting displayed notifications */));
 }
 
 // static