Revert of Add browser tests for FetchEvent.request.mode/credentials in the service worker

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}

[horo, 2016-02-25 06:43:22]

Created Revert of Add browser tests for FetchEvent.request.mode/credentials in
the service worker

[commit-bot: I haz the power, 2016-02-25 06:43:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739643002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739643002/1

[nhiroki, 2016-02-25 06:51:39]

> Reason for revert:
> https://codereview.chromium.org/1665533003/ introduced a security bug.

Did the bug fail these tests? Why don't you just disable failing tests?

[nhiroki, 2016-02-25 06:53:18]

On 2016/02/25 06:51:39, nhiroki wrote:
> > Reason for revert:
> > https://codereview.chromium.org/1665533003/ introduced a security bug.
> 
> Did the bug fail these tests? Why don't you just disable failing tests?

Ah, are you going to revert the base CL
(https://codereview.chromium.org/1665533003/), right?

[commit-bot: I haz the power, 2016-02-25 07:02:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739643002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739643002/1

[horo, 2016-02-25 07:13:32]

On 2016/02/25 06:53:18, nhiroki wrote:
> On 2016/02/25 06:51:39, nhiroki wrote:
> > > Reason for revert:
> > > https://codereview.chromium.org/1665533003/ introduced a security bug.
> > 
> > Did the bug fail these tests? Why don't you just disable failing tests?
> 
> Ah, are you going to revert the base CL
> (https://codereview.chromium.org/1665533003/), right?

yes. see crbug.com/589740

[nhiroki, 2016-02-25 07:16:20]

On 2016/02/25 07:13:32, horo wrote:
> On 2016/02/25 06:53:18, nhiroki wrote:
> > On 2016/02/25 06:51:39, nhiroki wrote:
> > > > Reason for revert:
> > > > https://codereview.chromium.org/1665533003/ introduced a security bug.
> > > 
> > > Did the bug fail these tests? Why don't you just disable failing tests?
> > 
> > Ah, are you going to revert the base CL
> > (https://codereview.chromium.org/1665533003/), right?
> 
> yes. see crbug.com/589740

I see. Thank you.

[commit-bot: I haz the power, 2016-02-25 07:32:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739643002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739643002/1

[commit-bot: I haz the power, 2016-02-25 08:02:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739643002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739643002/1

[commit-bot: I haz the power, 2016-02-25 08:12:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-25 08:13:22]

Description was changed from

==========
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...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=576534, 543895
==========

to

==========
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...
# 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}
==========

[commit-bot: I haz the power, 2016-02-25 08:13:23]

Patchset 1 (id:??) landed as
https://crrev.com/50ba0ec970300e054957e23e547a2a0d6759afc8
Cr-Commit-Position: refs/heads/master@{#377532}

[horo, 2016-03-08 23:36:31]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1775143003/ by horo@chromium.org.

The reason for reverting is: Bug fixed cl landed:
https://codereview.chromium.org/1762893002/.