Add LayoutTests for the fixed multithread GC bug of ServiceWorker Navigation Preload

Add LayoutTests for the fixed multithread GC bug of ServiceWorker Navigation Preload

BUG=702527
Review-Url: https://codereview.chromium.org/2764293002
Cr-Commit-Position: refs/heads/master@{#459348}
Committed: https://chromium.googlesource.com/chromium/src/+/85f49a916b4835ec93379a0eaeb4f125b66f6270

[horo, 2017-03-22 11:33:39]

Patchset #1 (id:1) has been deleted

[horo, 2017-03-22 11:33:48]

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

[commit-bot: I haz the power, 2017-03-22 11:33:57]

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-22 12:51:07]

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

[commit-bot: I haz the power, 2017-03-22 12:51:07]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[horo, 2017-03-22 13:51:03]

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

[commit-bot: I haz the power, 2017-03-22 13:51:11]

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-22 16:18:43]

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

[commit-bot: I haz the power, 2017-03-22 16:18:44]

Dry run: This issue passed the CQ dry run.

[horo, 2017-03-23 05:33:24]

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

[horo, 2017-03-23 05:33:25]

falken@
Could you please review this?

[falken, 2017-03-23 06:19:02]

lgtm. the sleeps are unfortunate but seem unavoidable unless Oilpan team knows
of a better way to exercise this crash fix in a test.

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/navigation-preload-after-gc.html
(right):

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/navigation-preload-after-gc.html:9:
// This test simulate the flowing situation:
nit: "simulates", "following"

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/preload-response-after-gc.html
(right):

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/preload-response-after-gc.html:9:
// This test simulate the flowing situation:
ditto

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/preload-response-after-gc.html:33:
// Wait for a while to ensure that the SW recieves the response.
nit: receives

[horo, 2017-03-23 06:46:39]

Thank you.

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/navigation-preload-after-gc.html
(right):

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/navigation-preload-after-gc.html:9:
// This test simulate the flowing situation:
On 2017/03/23 06:19:01, falken wrote:
> nit: "simulates", "following"

Done.

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/preload-response-after-gc.html
(right):

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/preload-response-after-gc.html:9:
// This test simulate the flowing situation:
On 2017/03/23 06:19:01, falken wrote:
> ditto

Done.

https://codereview.chromium.org/2764293002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/serviceworker/navigation-preload/chromium/preload-response-after-gc.html:33:
// Wait for a while to ensure that the SW recieves the response.
On 2017/03/23 06:19:01, falken wrote:
> nit: receives

Done.

[horo, 2017-03-23 06:47:03]

horo@chromium.org changed reviewers:
+ haraken@chromium.org

[horo, 2017-03-23 06:47:04]

haraken@
Could you please review this?

[haraken, 2017-03-23 07:05:02]

LGTM

[horo, 2017-03-24 00:45:54]

The CQ bit was checked by horo@chromium.org

[horo, 2017-03-24 00:45:54]

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/2764293002/#ps60001
(title: "incorporated falken's comment")

[commit-bot: I haz the power, 2017-03-24 00:46:22]

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-24 03:03:46]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490316354381260,
"parent_rev": "cb54fdec32a2b472df2a2d3e81d0180e41323273", "commit_rev":
"85f49a916b4835ec93379a0eaeb4f125b66f6270"}

[commit-bot: I haz the power, 2017-03-24 03:05:43]

Description was changed from

==========
Add LayoutTests for the fixed multithread GC bug of ServiceWorker Navigation
Preload

BUG=702527
==========

to

==========
Add LayoutTests for the fixed multithread GC bug of ServiceWorker Navigation
Preload

BUG=702527

Review-Url: https://codereview.chromium.org/2764293002
Cr-Commit-Position: refs/heads/master@{#459348}
Committed:
https://chromium.googlesource.com/chromium/src/+/85f49a916b4835ec93379a0eaeb4...
==========

[commit-bot: I haz the power, 2017-03-24 03:05:45]

Committed patchset #3 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/85f49a916b4835ec93379a0eaeb4...