DescriptionReland of Add browser tests for FetchEvent.request.mode/credentials in the service worker (patchset #1 id:1 of https://codereview.chromium.org/1739643002/ )
Reason for revert:
Bug fixed cl landed: https://codereview.chromium.org/1762893002/
Original issue's description:
> Revert of Add browser tests for FetchEvent.request.mode/credentials in the service worker (patchset #5 id:120001 of https://codereview.chromium.org/1665453003/ )
>
> Reason for revert:
> https://codereview.chromium.org/1665533003/ introduced a security bug.
>
> Original issue's description:
> > Add browser tests for FetchEvent.request.mode/credentials in the service worker
> >
> > This CL adds browser_tests to check the fetch request mode and
> > credentials mode in the service worker for the follwing requests.
> >
> > - <embed src="test.pdf">
> > Fetched in MimeHandlerViewContainer::OnReady().
> > - <link rel="manifest" href="manifest.json">
> > Fetched in ManifestFetcher::Start().
> > - PPAPI's pp::URLLoader.
> > Fetched in PepperURLLoaderHost::InternalOnHostMsgOpen().
> >
> > This CL depends on https://codereview.chromium.org/1665533003/.
> >
> > BUG=576534, 543895
> >
> > Committed: https://crrev.com/bcc7a2056bac1159aa480a8fb9355da98f84424c
> > Cr-Commit-Position: refs/heads/master@{#376704}
>
> TBR=jochen@chromium.org,bradnelson@chromium.org,bradnelson@google.com,nhiroki@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=576534, 543895
>
> Committed: https://crrev.com/50ba0ec970300e054957e23e547a2a0d6759afc8
> Cr-Commit-Position: refs/heads/master@{#377532}
TBR=jochen@chromium.org,bradnelson@chromium.org,bradnelson@google.com,nhiroki@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=576534, 543895
Committed: https://crrev.com/61f3f4f037aca857735769cda129e18e38e8853e
Cr-Commit-Position: refs/heads/master@{#380014}
Patch Set 1 #
Messages
Total messages: 8 (1 generated)
|