[test] Fix merging of test outcomes

[test] Fix merging of test outcomes

BUG=v8:5238
Committed: https://crrev.com/11d0d16ea93cfae2a86b5beb4a74fdf8e3c8bd4d
Cr-Commit-Position: refs/heads/master@{#38362}

[Michael Achenbach, 2016-08-05 08:59:10]

Description was changed from

==========
[test] Fix merging of test outcomes

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

to

==========
[test] Fix merging of test outcomes

BUG=v8:5238
==========

[Michael Achenbach, 2016-08-05 08:59:16]

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

[commit-bot: I haz the power, 2016-08-05 08:59:19]

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

[Michael Achenbach, 2016-08-05 09:02:06]

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

[Michael Achenbach, 2016-08-05 09:02:06]

PTAL. This stems from |= mutating the left hand object here:
https://cs.chromium.org/chromium/src/v8/tools/testrunner/local/testsuite.py?q...

We could also defensively use everywhere:
a = a | b
And also make all sets frozensets. Not sure if that could have a performance
impact.

[Michael Achenbach, 2016-08-05 09:02:41]

https://codereview.chromium.org/2216293002/diff/1/tools/testrunner/local/test...
File tools/testrunner/local/testsuite_unittest.py (right):

https://codereview.chromium.org/2216293002/diff/1/tools/testrunner/local/test...
tools/testrunner/local/testsuite_unittest.py:54: suite.tests = [
Btw: This test change reveals the problem.

[commit-bot: I haz the power, 2016-08-05 09:20:26]

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

[commit-bot: I haz the power, 2016-08-05 09:20:27]

Dry run: This issue passed the CQ dry run.

[Jakob Kummerow, 2016-08-05 09:33:45]

lgtm

[tandrii(chromium), 2016-08-05 09:36:31]

lgtm

[Michael Achenbach, 2016-08-05 09:38:35]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-08-05 09:38:39]

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, 2016-08-05 09:40:23]

Description was changed from

==========
[test] Fix merging of test outcomes

BUG=v8:5238
==========

to

==========
[test] Fix merging of test outcomes

BUG=v8:5238
==========

[commit-bot: I haz the power, 2016-08-05 09:40:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-05 09:43:38]

Description was changed from

==========
[test] Fix merging of test outcomes

BUG=v8:5238
==========

to

==========
[test] Fix merging of test outcomes

BUG=v8:5238

Committed: https://crrev.com/11d0d16ea93cfae2a86b5beb4a74fdf8e3c8bd4d
Cr-Commit-Position: refs/heads/master@{#38362}
==========

[commit-bot: I haz the power, 2016-08-05 09:43:39]

Patchset 1 (id:??) landed as
https://crrev.com/11d0d16ea93cfae2a86b5beb4a74fdf8e3c8bd4d
Cr-Commit-Position: refs/heads/master@{#38362}