Make GetDisplayedNotifications asynchronous.

content/browser/notifications/platform_notification_context_impl.h

 // Copyright 2015 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 CONTENT_BROWSER_NOTIFICATIONS_PLATFORM_NOTIFICATION_CONTEXT_IMPL_H_
 #define CONTENT_BROWSER_NOTIFICATIONS_PLATFORM_NOTIFICATION_CONTEXT_IMPL_H_
 
 #include <stdint.h>
 #include <memory>
 #include <set>
@@ skipping 83 matching lines @@
   // ServiceWorkerContextObserver implementation.
   void OnRegistrationDeleted(int64_t registration_id,
                              const GURL& pattern) override;
   void OnStorageWiped() override;
 
  private:
   friend class PlatformNotificationContextTest;
 
   ~PlatformNotificationContextImpl() override;
 
-  void InitializeOnIO();
+  void DidGetNotificationsOnUI(
+      std::unique_ptr<std::set<std::string>> displayed_notifications,
+      bool notification_synchronization_supported);
+  void InitializeOnIO(
+      std::unique_ptr<std::set<std::string>> displayed_notifications,
+      bool notification_synchronization_supported);
   void ShutdownOnIO();
   void CreateServiceOnIO(
       int render_process_id,
       ResourceContext* resource_context,
       mojo::InterfaceRequest<blink::mojom::NotificationService> request);
 
   // Initializes the database if neccesary. Must be called on the IO thread.
   // |success_closure| will be invoked on a the |task_runner_| thread when
   // everything is available, or |failure_closure_| will be invoked on the
   // IO thread when initialization fails.
   void LazyInitialize(const base::Closure& success_closure,
                       const base::Closure& failure_closure);
 
   // Opens the database. Must be called on the |task_runner_| thread. When the
   // database has been opened, |success_closure| will be invoked on the task
   // thread, otherwise |failure_closure_| will be invoked on the IO thread.
   void OpenDatabase(const base::Closure& success_closure,
                     const base::Closure& failure_closure);
 
   // Actually reads the notification data from the database. Must only be
   // called on the |task_runner_| thread. |callback| will be invoked on the
   // IO thread when the operation has completed.
   void DoReadNotificationData(const std::string& notification_id,
                               const GURL& origin,
                               const ReadResultCallback& callback);
 
   // Updates the database (and the result callback) based on
   // |displayed_notifications|
-  // if |sync_supported|. Called on the IO thread.
-  void SynchronizeDisplayedNotificationsForServiceWorkerRegistration(
+  // if |sync_supported|.
+  void SynchronizeDisplayedNotificationsForServiceWorkerRegistrationOnUI(
       const GURL& origin,
       int64_t service_worker_registration_id,
       const ReadAllResultCallback& callback,
+      std::unique_ptr<std::set<std::string>> displayed_notifications,
+      bool sync_supported);
+
+  // Updates the database (and the result callback) based on
+  // |displayed_notifications|
+  // if |sync_supported|.

[Peter Beverloo, 2017/03/17 15:46:04]

nit: fix weird line break (also on line 137)

[Miguel Garcia, 2017/03/20 14:11:21]

Done.

+  void SynchronizeDisplayedNotificationsForServiceWorkerRegistrationOnIO(
+      const GURL& origin,
+      int64_t service_worker_registration_id,
+      const ReadAllResultCallback& callback,
       std::unique_ptr<std::set<std::string>> displayed_notifications,
       bool sync_supported);
 
   // Actually reads all notification data from the database. Must only be
   // called on the |task_runner_| thread. |callback| will be invoked on the
   // IO thread when the operation has completed.
   void DoReadAllNotificationDataForServiceWorkerRegistration(
       const GURL& origin,
       int64_t service_worker_registration_id,
       const ReadAllResultCallback& callback,
@@ skipping 48 matching lines @@
   // The notification services are owned by the platform context, and will be
   // removed when either this class is destroyed or the Mojo pipe disconnects.
   std::vector<std::unique_ptr<BlinkNotificationServiceImpl>> services_;
 
   DISALLOW_COPY_AND_ASSIGN(PlatformNotificationContextImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_NOTIFICATIONS_PLATFORM_NOTIFICATION_CONTEXT_IMPL_H_