Do not compile tests for different browsers separately

Do not compile tests for different browsers separately

Currently we recompile tests with dart2js for all browsers. There is no
need to do these compilations for all browsers, since the output should be
exactly the same.

R=ricow@google.com

Committed: https://code.google.com/p/dart/source/detail?r=24402

[ricow1, 2013-06-25 13:10:48]

LGTM, I think we should a a comment somewhere to actually explain _why_ we do
this. Otherwise somebody (as in one of us when we forgot why in 6 months) will
remove this again.
I would also get Peter to take a look

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_runne...
File tools/testing/dart/test_runner.dart (right):

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_runne...
tools/testing/dart/test_runner.dart:201:
toFile.writeAsBytesSync(fromFile.readAsBytesSync());
add small comment stating why we do this sync

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_suite...
File tools/testing/dart/test_suite.dart (right):

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_suite...
tools/testing/dart/test_suite.dart:994: // FIXME: This might be an issue
No fixmes

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_suite...
tools/testing/dart/test_suite.dart:1118: Command _compileCommand(String
inputFile, String outputFile, String
move String down

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_suite...
tools/testing/dart/test_suite.dart:1182: "$checked$minified$csp";
fits one line?

[kustermann, 2013-06-25 13:20:11]

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_runne...
File tools/testing/dart/test_runner.dart (right):

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_runne...
tools/testing/dart/test_runner.dart:201:
toFile.writeAsBytesSync(fromFile.readAsBytesSync());
On 2013/06/25 13:10:48, ricow1 wrote:
> add small comment stating why we do this sync

Done.

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_suite...
File tools/testing/dart/test_suite.dart (right):

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_suite...
tools/testing/dart/test_suite.dart:994: // FIXME: This might be an issue
On 2013/06/25 13:10:48, ricow1 wrote:
> No fixmes

Done.

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_suite...
tools/testing/dart/test_suite.dart:1118: Command _compileCommand(String
inputFile, String outputFile, String
On 2013/06/25 13:10:48, ricow1 wrote:
> move String down

Done.

https://codereview.chromium.org/17570018/diff/1/tools/testing/dart/test_suite...
tools/testing/dart/test_suite.dart:1182: "$checked$minified$csp";
On 2013/06/25 13:10:48, ricow1 wrote:
> fits one line?

Done.

[kustermann, 2013-06-25 13:25:19]

Committed patchset #2 manually as r24402 (presubmit successful).