Remove duplicate service wrkr "shared worker" test

Remove duplicate service wrkr "shared worker" test

This test is identical to the version available in the Web Platform
Tests project and may therefore be removed without impacting coverage.

BUG=688116
R=falken@chromium.org

Review-Url: https://codereview.chromium.org/2897113002
Cr-Commit-Position: refs/heads/master@{#475066}
Committed: https://chromium.googlesource.com/chromium/src/+/35067fccc9e3904e01a9a650fc664ff8170446f2

[mike3, 2017-05-23 18:02:56]

Here's the proof :)

https://gist.github.com/jugglinmike/040cc59ef51bf30a96185dff35a70ce5

[falken, 2017-05-26 01:21:22]

lgtm

[falken, 2017-05-26 01:21:31]

The CQ bit was checked by falken@chromium.org

[commit-bot: I haz the power, 2017-05-26 01:23:04]

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-05-26 03:09:07]

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

[commit-bot: I haz the power, 2017-05-26 03:09:08]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[mike3, 2017-05-26 14:38:28]

On 2017/05/26 03:09:08, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

Thanks, Matt. This patch failed in CI, but I have never been able to interpret
the provided logs. (See [1] for my latest attempt to do so.) I expect I'm just
missing some relevant link; could you suggest which log file ought to have
information about the specific assertion failures?

[1] https://codereview.chromium.org/2897083002/#msg10

[falken, 2017-05-26 15:01:51]

On 2017/05/26 14:38:28, mike3 wrote:
> On 2017/05/26 03:09:08, commit-bot: I haz the power wrote:
> > Try jobs failed on following builders:
> >   win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
> >
>
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)
> 
> Thanks, Matt. This patch failed in CI, but I have never been able to interpret
> the provided logs. (See [1] for my latest attempt to do so.) I expect I'm just
> missing some relevant link; could you suggest which log file ought to have
> information about the specific assertion failures?
> 
> [1] https://codereview.chromium.org/2897083002/#msg10

Ah sorry. Look for "archive_webkit_tests_results" then "layout_test_results"
under it. That'll goes to a page like this which displays the test output:
https://storage.googleapis.com/chromium-layout-test-archives/linux_chromium_r...

[mike3, 2017-05-26 16:54:41]

On 2017/05/26 15:01:51, falken wrote:
> Ah sorry. Look for "archive_webkit_tests_results" then "layout_test_results"
> under it. That'll goes to a page like this which displays the test output:
>
https://storage.googleapis.com/chromium-layout-test-archives/linux_chromium_r...

Ahah, there we go! Thanks for the tip, Matt. Now I can see that I was too
aggressive with file deletion in this patch--the shared "resource" file
`simple-intercept-worker.js` is also required by the
`unregister-controller.html` test. I've re-introduced the resource file here.

As it happens, I have another CL open to remove `unregister-controller.html`:

https://codereview.chromium.org/2904123002/

When this patch lands, I will rebase that CL and delete
`simple-intercept-worker.js` once and for all.

[mike3, 2017-05-26 16:54:45]

The CQ bit was checked by mike@mikepennisi.com

[mike3, 2017-05-26 16:54:45]

The patchset sent to the CQ was uploaded after l-g-t-m from falken@chromium.org
Link to the patchset: https://codereview.chromium.org/2897113002/#ps20001
(title: "Re-introduce shared resource file")

[commit-bot: I haz the power, 2017-05-26 16:55:23]

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-05-26 18:28:30]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495817685027420,
"parent_rev": "6f0095b330f52533f1cad17bb6df7f4480eadf0e", "commit_rev":
"35067fccc9e3904e01a9a650fc664ff8170446f2"}

[commit-bot: I haz the power, 2017-05-26 18:28:41]

Description was changed from

==========
Remove duplicate service wrkr "shared worker" test

This test is identical to the version available in the Web Platform
Tests project and may therefore be removed without impacting coverage.

BUG=688116
R=falken@chromium.org
==========

to

==========
Remove duplicate service wrkr "shared worker" test

This test is identical to the version available in the Web Platform
Tests project and may therefore be removed without impacting coverage.

BUG=688116
R=falken@chromium.org

Review-Url: https://codereview.chromium.org/2897113002
Cr-Commit-Position: refs/heads/master@{#475066}
Committed:
https://chromium.googlesource.com/chromium/src/+/35067fccc9e3904e01a9a650fc66...
==========

[commit-bot: I haz the power, 2017-05-26 18:28:43]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/35067fccc9e3904e01a9a650fc66...