DescriptionRevert of ServiceWorker: Support includeUncontrolled option in clients.matchAll() (1/2, chromium) (patchset #4 id:120001 of https://codereview.chromium.org/1007133002/)
Reason for revert:
Broke http/tests/serviceworker/chromium/clients-openwindow.html
http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux/builds/45800
Original issue's description:
> ServiceWorker: Support includeUncontrolled option in clients.matchAll() (1/2, chromium)
>
> * Add SWContextCore::GetProviderHostByClientID() method
> * Add SWContextCore::GetProviderHostIterator() which takes predicate
> * Make ServiceWorkerVersion::OnGetClients return uncontrolled host when specified
> * Make ServiceWorkerVersion::On{PostMessage,Focus} work for uncontrolled hosts
> (i.e. now ServiceWorker can have clients that are not controlled by it)
>
> Spec links:
> https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#serviceworker-client-query-options-dictionary
> https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#clients-getall
>
> 1/2: THIS PATCH
> 2/2: blink patch (layout tests will be added here)
>
> BUG=460415
> TEST=ServiceWorkerContextTest.GetProviderHostIterator
> TEST=layout tests will be added later
>
> Committed: https://crrev.com/04848e8f57c823b674b0a3e03047e281fae44bfd
> Cr-Commit-Position: refs/heads/master@{#321722}
TBR=nhiroki@chromium.org,kinuko@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=460415
Committed: https://crrev.com/dec2cd69cca6346089eee3c2110d9d3518e3b3d4
Cr-Commit-Position: refs/heads/master@{#321749}
Patch Set 1 #
Created: 5 years, 9 months ago
(Patch set is too large to download)
Messages
Total messages: 7 (0 generated)
|