Fetch API: Remove expectation files of request.js

Fetch API: Remove expectation files of request.js

Fixes the "duplicate test names" error for tests in request.js, 
after which all test cases in request.js pass without warnings 
or errors. So the expectation files can be safely removed. 

BUG=624278
Review-Url: https://codereview.chromium.org/2778183002
Cr-Commit-Position: refs/heads/master@{#460257}
Committed: https://chromium.googlesource.com/chromium/src/+/e589313cfc0a5f177668275af93feedb046f783f

[yiyix, 2017-03-28 12:07:01]

Description was changed from

==========
fix tests

BUG=
==========

to

==========
Fix tests in request.js

Now all tests in request.js are expected to pass, so the expectation 
file can be removed safely. 

BUG=624278
==========

[yiyix, 2017-03-28 12:07:06]

The CQ bit was checked by yiyix@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-28 12:07:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-28 13:11:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-28 13:11:09]

Dry run: This issue passed the CQ dry run.

[yiyix, 2017-03-29 01:03:03]

Description was changed from

==========
Fix tests in request.js

Now all tests in request.js are expected to pass, so the expectation 
file can be removed safely. 

BUG=624278
==========

to

==========
Remove expectation files of request.js

All tests in request.js are expected to pass now, so the expectation 
files can be removed safely. 

BUG=624278
==========

[yiyix, 2017-03-29 01:04:16]

yiyix@chromium.org changed reviewers:
+ falken@chromium.org

[yiyix, 2017-03-29 01:06:31]

@falken, could you please review this patch? Thank you very much.

[falken, 2017-03-29 01:06:58]

lgtm

It'd be a bit more explanatory if the CL description mentioned that this is
fixing the "duplicate test names" error, after which the expectation files
contain no errors and can be removed.

[falken, 2017-03-29 01:07:23]

Also could add "Fetch API: " or something to the first line.

[yiyix, 2017-03-29 01:09:38]

Description was changed from

==========
Remove expectation files of request.js

All tests in request.js are expected to pass now, so the expectation 
files can be removed safely. 

BUG=624278
==========

to

==========
Fetch API: Remove expectation files of request.js

All tests in request.js are expected to pass now, so the expectation 
files can be removed safely. 

BUG=624278
==========

[yiyix, 2017-03-29 01:09:53]

The CQ bit was checked by yiyix@chromium.org

[yiyix, 2017-03-29 01:09:59]

The CQ bit was unchecked by yiyix@chromium.org

[yiyix, 2017-03-29 01:15:21]

Description was changed from

==========
Fetch API: Remove expectation files of request.js

All tests in request.js are expected to pass now, so the expectation 
files can be removed safely. 

BUG=624278
==========

to

==========
Fetch API: Remove expectation files of request.js

Fixes the "duplicate test names" error for tests in request.js, 
after which all test cases in request.js pass without warnings 
or errors. So the expectation files can be removed safely. 

BUG=624278
==========

[yiyix, 2017-03-29 01:16:31]

Description was changed from

==========
Fetch API: Remove expectation files of request.js

Fixes the "duplicate test names" error for tests in request.js, 
after which all test cases in request.js pass without warnings 
or errors. So the expectation files can be removed safely. 

BUG=624278
==========

to

==========
Fetch API: Remove expectation files of request.js

Fixes the "duplicate test names" error for tests in request.js, 
after which all test cases in request.js pass without warnings 
or errors. So the expectation files can be safely removed. 

BUG=624278
==========

[yiyix, 2017-03-29 01:16:47]

On 2017/03/29 01:07:23, falken wrote:
> Also could add "Fetch API: " or something to the first line.

Updated, thank you.

[yiyix, 2017-03-29 01:16:58]

The CQ bit was checked by yiyix@chromium.org

[commit-bot: I haz the power, 2017-03-29 01:17:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-29 01:22:54]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490750218742370, "parent_rev":
"b663358c89e05334e8cfff0f1f855fe4c72c192a", "commit_rev":
"e589313cfc0a5f177668275af93feedb046f783f"}

[commit-bot: I haz the power, 2017-03-29 01:23:42]

Description was changed from

==========
Fetch API: Remove expectation files of request.js

Fixes the "duplicate test names" error for tests in request.js, 
after which all test cases in request.js pass without warnings 
or errors. So the expectation files can be safely removed. 

BUG=624278
==========

to

==========
Fetch API: Remove expectation files of request.js

Fixes the "duplicate test names" error for tests in request.js, 
after which all test cases in request.js pass without warnings 
or errors. So the expectation files can be safely removed. 

BUG=624278

Review-Url: https://codereview.chromium.org/2778183002
Cr-Commit-Position: refs/heads/master@{#460257}
Committed:
https://chromium.googlesource.com/chromium/src/+/e589313cfc0a5f177668275af93f...
==========

[commit-bot: I haz the power, 2017-03-29 01:23:43]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e589313cfc0a5f177668275af93f...