[ServiceWorker] waitUntil() should wait until all promises got resolved/rejected.

third_party/WebKit/Source/web/ServiceWorkerGlobalScopeProxy.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 272 matching lines @@
     // onNavigationPreloadError() will be called.
     pending_preload_fetch_events_.insert(fetch_event_id, fetch_event);
   }
   wait_until_observer->WillDispatchEvent();
   respond_with_observer->WillDispatchEvent();
   DispatchEventResult dispatch_result =
       WorkerGlobalScope()->DispatchEvent(fetch_event);
   respond_with_observer->DidDispatchEvent(dispatch_result);
   // false is okay because waitUntil for fetch event doesn't care about the
   // promise rejection or an uncaught runtime script error.
-  wait_until_observer->DidDispatchEvent(false /* errorOccurred */);
+  wait_until_observer->DidDispatchEvent(false /* event_dispatch_failed */);
 }
 
 void ServiceWorkerGlobalScopeProxy::OnNavigationPreloadResponse(
     int fetch_event_id,
     std::unique_ptr<WebURLResponse> response,
     std::unique_ptr<WebDataConsumerHandle> data_consume_handle) {
   auto it = pending_preload_fetch_events_.find(fetch_event_id);
   DCHECK(it != pending_preload_fetch_events_.end());
   FetchEvent* fetch_event = it->value.Get();
   DCHECK(fetch_event);
@@ skipping 72 matching lines @@
       WorkerGlobalScope()->ScriptController()->GetScriptState(),
       EventTypeNames::foreignfetch, event_init, respond_with_observer,
       wait_until_observer);
   wait_until_observer->WillDispatchEvent();
   respond_with_observer->WillDispatchEvent();
   DispatchEventResult dispatch_result =
       WorkerGlobalScope()->DispatchEvent(fetch_event);
   respond_with_observer->DidDispatchEvent(dispatch_result);
   // false is okay because waitUntil for foreign fetch event doesn't care
   // about the promise rejection or an uncaught runtime script error.
-  wait_until_observer->DidDispatchEvent(false /* errorOccurred */);
+  wait_until_observer->DidDispatchEvent(false /* event_dispatch_failed */);
 }
 
 void ServiceWorkerGlobalScopeProxy::DispatchInstallEvent(int event_id) {
   WaitUntilObserver* observer = WaitUntilObserver::Create(
       WorkerGlobalScope(), WaitUntilObserver::kInstall, event_id);
   Event* event = InstallEvent::Create(
       EventTypeNames::install, ExtendableEventInit(), event_id, observer);
   WorkerGlobalScope()->DispatchExtendableEvent(event, observer);
 }
 
@@ skipping 73 matching lines @@
           web_app_request),
       respond_with_observer, wait_until_observer);
 
   wait_until_observer->WillDispatchEvent();
   respond_with_observer->WillDispatchEvent();
   DispatchEventResult dispatch_result =
       WorkerGlobalScope()->DispatchEvent(event);
   respond_with_observer->DidDispatchEvent(dispatch_result);
   // false is okay because waitUntil for payment request event doesn't care
   // about the promise rejection or an uncaught runtime script error.
-  wait_until_observer->DidDispatchEvent(false /* errorOccurred */);
+  wait_until_observer->DidDispatchEvent(false /* event_dispatch_failed */);
 }
 
 bool ServiceWorkerGlobalScopeProxy::HasFetchEventHandler() {
   DCHECK(worker_global_scope_);
   return worker_global_scope_->HasEventListeners(EventTypeNames::fetch);
 }
 
 void ServiceWorkerGlobalScopeProxy::CountFeature(UseCounter::Feature feature) {
   Client().CountFeature(static_cast<uint32_t>(feature));
 }
@@ skipping 121 matching lines @@
   return *document_;
 }
 
 ServiceWorkerGlobalScope* ServiceWorkerGlobalScopeProxy::WorkerGlobalScope()
     const {
   DCHECK(worker_global_scope_);
   return worker_global_scope_;
 }
 
 }  // namespace blink