Fix two tests in http/tests/serviceworker/ to handle the correct behavior of undefined in arrays, a…

Fix two tests in http/tests/serviceworker/ to handle the correct behavior of undefined in arrays, and suppress them until the fix relands.

These tests currently depend on the incorrect behavior wherein "undefined"
elements are sometimes removed from arrays (when they should be present, but
have the value undefined).

TestExpectations lines will be removed once the V8 change re-rolls.

V8 revert was:
https://chromium.googlesource.com/v8/v8/+/b861a8400991976e6fd7d7abc9843e4fe7dc576e

BUG=chromium:665820
Review-Url: https://codereview.chromium.org/2662953002
Cr-Commit-Position: refs/heads/master@{#447152}
Committed: https://chromium.googlesource.com/chromium/src/+/507c8a3ed5bc7686ac0e7d9a6b9db0d0d413a63b

[jbroman, 2017-01-30 19:55:54]

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

[commit-bot: I haz the power, 2017-01-30 19:56:23]

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

[jbroman, 2017-01-30 19:56:36]

Description was changed from

==========
Fix two tests in http/tests/serviceworker/ to handle the correct behavior of
undefined in arrays, and suppress them until the fix relands.

These tests currently depend on the incorrect behavior wherein "undefined"
elements are sometimes removed from arrays (when they should be present, but
have the value undefined).

TestExpectations lines will be removed once the V8 change re-rolls.

BUG=chromium:665820
==========

to

==========
Fix two tests in http/tests/serviceworker/ to handle the correct behavior of
undefined in arrays, and suppress them until the fix relands.

These tests currently depend on the incorrect behavior wherein "undefined"
elements are sometimes removed from arrays (when they should be present, but
have the value undefined).

TestExpectations lines will be removed once the V8 change re-rolls.

V8 revert was:
https://chromium.googlesource.com/v8/v8/+/b861a8400991976e6fd7d7abc9843e4fe7d...

BUG=chromium:665820
==========

[jbroman, 2017-01-30 20:10:26]

jbroman@chromium.org changed reviewers:
+ nhiroki@chromium.org

[jbroman, 2017-01-30 20:10:27]

At the sending end, these elements are present; this fixes the tests to expect
the correct behavior (and suppresses the failures until the fix can roll into
Chromium again).

[nhiroki, 2017-01-30 21:08:39]

LGTM!

[commit-bot: I haz the power, 2017-01-30 21:09:39]

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

[commit-bot: I haz the power, 2017-01-30 21:09:39]

Dry run: 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_...)

[jbroman, 2017-01-30 22:01:48]

The CQ bit was checked by jbroman@chromium.org

[jbroman, 2017-01-30 22:01:54]

The CQ bit was unchecked by jbroman@chromium.org

[jbroman, 2017-01-30 22:13:37]

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

[commit-bot: I haz the power, 2017-01-30 22:14:35]

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-01-30 23:38:17]

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

[commit-bot: I haz the power, 2017-01-30 23:38:18]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jbroman, 2017-01-30 23:44:14]

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

[commit-bot: I haz the power, 2017-01-30 23:45:01]

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

[jbroman, 2017-01-30 23:46:22]

The CQ bit was unchecked by jbroman@chromium.org

[jbroman, 2017-01-30 23:46:24]

The CQ bit was checked by jbroman@chromium.org

[jbroman, 2017-01-30 23:46:24]

The patchset sent to the CQ was uploaded after l-g-t-m from nhiroki@chromium.org
Link to the patchset: https://codereview.chromium.org/2662953002/#ps40001
(title: "whoops, fix TestExpectations file again")

[commit-bot: I haz the power, 2017-01-30 23:46:34]

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-01-31 01:20:35]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485819984313400,
"parent_rev": "af3cf10b28c33140c430fb10a0f3ed301d9c84f9", "commit_rev":
"507c8a3ed5bc7686ac0e7d9a6b9db0d0d413a63b"}

[commit-bot: I haz the power, 2017-01-31 01:21:06]

Description was changed from

==========
Fix two tests in http/tests/serviceworker/ to handle the correct behavior of
undefined in arrays, and suppress them until the fix relands.

These tests currently depend on the incorrect behavior wherein "undefined"
elements are sometimes removed from arrays (when they should be present, but
have the value undefined).

TestExpectations lines will be removed once the V8 change re-rolls.

V8 revert was:
https://chromium.googlesource.com/v8/v8/+/b861a8400991976e6fd7d7abc9843e4fe7d...

BUG=chromium:665820
==========

to

==========
Fix two tests in http/tests/serviceworker/ to handle the correct behavior of
undefined in arrays, and suppress them until the fix relands.

These tests currently depend on the incorrect behavior wherein "undefined"
elements are sometimes removed from arrays (when they should be present, but
have the value undefined).

TestExpectations lines will be removed once the V8 change re-rolls.

V8 revert was:
https://chromium.googlesource.com/v8/v8/+/b861a8400991976e6fd7d7abc9843e4fe7d...

BUG=chromium:665820

Review-Url: https://codereview.chromium.org/2662953002
Cr-Commit-Position: refs/heads/master@{#447152}
Committed:
https://chromium.googlesource.com/chromium/src/+/507c8a3ed5bc7686ac0e7d9a6b9d...
==========

[commit-bot: I haz the power, 2017-01-31 01:21:07]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/507c8a3ed5bc7686ac0e7d9a6b9d...