[test] Remove dependent commands.

[test] Remove dependent commands.

BUG=

Committed: https://crrev.com/76876b9ae9079598347045021b72d5cbd706f791
Cr-Commit-Position: refs/heads/master@{#34428}

[Michael Achenbach, 2016-03-02 09:20:33]

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

[commit-bot: I haz the power, 2016-03-02 09:20:36]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1753803003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1753803003/1

[commit-bot: I haz the power, 2016-03-02 09:37:31]

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

[commit-bot: I haz the power, 2016-03-02 09:37:32]

Dry run: This issue passed the CQ dry run.

[Michael Achenbach, 2016-03-02 11:22:47]

machenbach@chromium.org changed reviewers:
+ jkummerow@chromium.org, yangguo@chromium.org

[Michael Achenbach, 2016-03-02 11:22:47]

machenbach@chromium.org changed reviewers:
+ jkummerow@chromium.org, yangguo@chromium.org

[Michael Achenbach, 2016-03-02 11:22:48]

PTAL

https://codereview.chromium.org/1753803003/diff/20001/test/cctest/test-serial...
File test/cctest/test-serialize.cc (right):

https://codereview.chromium.org/1753803003/diff/20001/test/cctest/test-serial...
test/cctest/test-serialize.cc:247: Serialize(isolate);
FYI: The old "Serialize" test doesn't call isolate->Dispose(), so I kept it like
that.

https://codereview.chromium.org/1753803003/diff/20001/test/cctest/test-serial...
test/cctest/test-serialize.cc:364: PartiallySerialize();
For all serialize/deserialize pairs: Do we want the serialize parts to be tested
in isolation, too? The code is also a strict prefix of the deserialize tests.

[Michael Achenbach, 2016-03-02 11:22:48]

PTAL

https://codereview.chromium.org/1753803003/diff/20001/test/cctest/test-serial...
File test/cctest/test-serialize.cc (right):

https://codereview.chromium.org/1753803003/diff/20001/test/cctest/test-serial...
test/cctest/test-serialize.cc:247: Serialize(isolate);
FYI: The old "Serialize" test doesn't call isolate->Dispose(), so I kept it like
that.

https://codereview.chromium.org/1753803003/diff/20001/test/cctest/test-serial...
test/cctest/test-serialize.cc:364: PartiallySerialize();
For all serialize/deserialize pairs: Do we want the serialize parts to be tested
in isolation, too? The code is also a strict prefix of the deserialize tests.

[Michael Achenbach, 2016-03-02 11:23:28]

FYI: Ran some tests on patch 2.

[Yang, 2016-03-02 11:40:28]

On 2016/03/02 11:23:28, Michael Achenbach wrote:
> FYI: Ran some tests on patch 2.

C++ part LGTM.

We should get rid of writing and reading the testing serialization file, and put
it into a buffer instead.

[Jakob Kummerow, 2016-03-02 12:04:25]

Python changes LGTM.

Yay!

[Michael Achenbach, 2016-03-02 12:51:53]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-03-02 12:51:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1753803003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1753803003/40001

[commit-bot: I haz the power, 2016-03-02 13:13:07]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-02 13:14:19]

Description was changed from

==========
[test] Remove dependent commands.

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

to

==========
[test] Remove dependent commands.

BUG=

Committed: https://crrev.com/76876b9ae9079598347045021b72d5cbd706f791
Cr-Commit-Position: refs/heads/master@{#34428}
==========

[commit-bot: I haz the power, 2016-03-02 13:14:20]

Patchset 3 (id:??) landed as
https://crrev.com/76876b9ae9079598347045021b72d5cbd706f791
Cr-Commit-Position: refs/heads/master@{#34428}