Remove fast/storage/serialized-script-value.html and serialized-script-value-collections.html.

Remove fast/storage/serialized-script-value.html and serialized-script-value-collections.html.

These are very sensitive to changes in the wire format, and the copmatibility
cases they are designed to test are covered by V8 and Blink unit tests.

This makes it easier to land changes such as this one:
  https://codereview.chromium.org/2658793004

BUG=686159
Review-Url: https://codereview.chromium.org/2654993009
Cr-Commit-Position: refs/heads/master@{#446749}
Committed: https://chromium.googlesource.com/chromium/src/+/679cd54a09a4ff192ea8bb3f5af32a73299ff0cd

[jbroman, 2017-01-27 18:49:35]

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

[commit-bot: I haz the power, 2017-01-27 18:50:04]

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-27 18:50:13]

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

[commit-bot: I haz the power, 2017-01-27 18:50:26]

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-27 18:50:37]

Description was changed from

==========
Remove fast/storage/serialized-script-value.html and
serialized-script-value-collections.html.

These are very sensitive to changes in the wire format, and the copmatibility
cases they are designed to test are covered by V8 and Blink unit tests.

BUG=686159
==========

to

==========
Remove fast/storage/serialized-script-value.html and
serialized-script-value-collections.html.

These are very sensitive to changes in the wire format, and the copmatibility
cases they are designed to test are covered by V8 and Blink unit tests.

This makes it easier to land changes such as this one:
  https://codereview.chromium.org/2658793004

BUG=686159
==========

[jbroman, 2017-01-27 18:52:00]

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

[jbroman, 2017-01-27 18:52:00]

Started seeing this because I wanted to send out an improvement to how 8-bit
strings are serialized, but this logic is on the V8 side and this layout test is
sensitive to how it's serialized (causing v8_linux_blink_rel failures that would
require a three-sided patch to resolve).

There are unit tests that cover this behaviour, so it seems reasonable to drop
these layout tests.

[jsbell, 2017-01-27 19:54:44]

lgtm

I notice a few other uses of internals.deserializeBuffer in tests (wasm, crypto)
should we be looking to remove those and the API itself?

[jbroman, 2017-01-27 20:12:02]

On 2017/01/27 at 19:54:44, jsbell wrote:
> lgtm
> 
> I notice a few other uses of internals.deserializeBuffer in tests (wasm,
crypto) should we be looking to remove those and the API itself?

deserializeBuffer isn't as much an issue (as we should be able to deserialize a
given input from the past), but what's problematic is serializing an object and
saying "the serialized output must equal this".

Selfishly, I use that internals API during development quite a bit.

[jbroman, 2017-01-27 20:12:07]

The CQ bit was unchecked by jbroman@chromium.org

[jbroman, 2017-01-27 20:12:08]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2017-01-27 20:12:35]

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-27 20:19:08]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485547928374120,
"parent_rev": "37b415b92e8873dad49d71be1c7562319926a3c5", "commit_rev":
"679cd54a09a4ff192ea8bb3f5af32a73299ff0cd"}

[commit-bot: I haz the power, 2017-01-27 20:19:38]

Description was changed from

==========
Remove fast/storage/serialized-script-value.html and
serialized-script-value-collections.html.

These are very sensitive to changes in the wire format, and the copmatibility
cases they are designed to test are covered by V8 and Blink unit tests.

This makes it easier to land changes such as this one:
  https://codereview.chromium.org/2658793004

BUG=686159
==========

to

==========
Remove fast/storage/serialized-script-value.html and
serialized-script-value-collections.html.

These are very sensitive to changes in the wire format, and the copmatibility
cases they are designed to test are covered by V8 and Blink unit tests.

This makes it easier to land changes such as this one:
  https://codereview.chromium.org/2658793004

BUG=686159

Review-Url: https://codereview.chromium.org/2654993009
Cr-Commit-Position: refs/heads/master@{#446749}
Committed:
https://chromium.googlesource.com/chromium/src/+/679cd54a09a4ff192ea8bb3f5af3...
==========

[commit-bot: I haz the power, 2017-01-27 20:19:39]

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