[WebAPKs] Plumb service worker scope to notifications

chrome/browser/push_messaging/push_messaging_notification_manager.cc

 // 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.
 
 #include "chrome/browser/push_messaging/push_messaging_notification_manager.h"
 
 #include <stddef.h>
 
 #include <bitset>
 
@@ skipping 333 matching lines @@
     const base::Closure& message_handled_closure,
     bool success,
     int64_t persistent_notification_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   if (!success) {
     DLOG(ERROR) << "Writing forced notification to database should not fail";
     message_handled_closure.Run();
     return;
   }
 
+  // Do not pass service worker scope. The origin will be used instead of the
+  // service worker scope to determine whether a notification should be
+  // attributed to a WebAPK on Android. This is OK because this code path is hit
+  // rarely.
   PlatformNotificationServiceImpl::GetInstance()->DisplayPersistentNotification(
-      profile_, persistent_notification_id, origin, notification_data,
-      NotificationResources());
+      profile_, persistent_notification_id, GURL() /* service_worker_scope */,
+      origin, notification_data, NotificationResources());
 
   message_handled_closure.Run();
 }