Don't pass an asset through a transformer that produces an error.

Don't pass an asset through a transformer that produces an error.

R=rnystrom@google.com
BUG=17415
Committed: https://code.google.com/p/dart/source/detail?r=33661

[Bob Nystrom, 2014-03-13 16:56:46]

https://codereview.chromium.org/195993005/diff/1/pkg/barback/test/package_gra...
File pkg/barback/test/package_graph/collisions_test.dart (left):

https://codereview.chromium.org/195993005/diff/1/pkg/barback/test/package_gra...
pkg/barback/test/package_graph/collisions_test.dart:15: test("errors if two
transformers output the same file", () {
Why are these tests deleted?

[nweiz, 2014-03-13 19:31:37]

https://codereview.chromium.org/195993005/diff/1/pkg/barback/test/package_gra...
File pkg/barback/test/package_graph/collisions_test.dart (left):

https://codereview.chromium.org/195993005/diff/1/pkg/barback/test/package_gra...
pkg/barback/test/package_graph/collisions_test.dart:15: test("errors if two
transformers output the same file", () {
On 2014/03/13 16:56:47, Bob Nystrom wrote:
> Why are these tests deleted?

This is a new file, collisions_test; these tests stayed in the old file,
errors_test. I split them out because errors_test was getting pretty big.

Now that I look at it a couple of these still have to do with collisions,
though, so I'll move them in here.

[Bob Nystrom, 2014-03-13 19:50:35]

On 2014/03/13 19:31:37, nweiz wrote:
>
https://codereview.chromium.org/195993005/diff/1/pkg/barback/test/package_gra...
> File pkg/barback/test/package_graph/collisions_test.dart (left):
> 
>
https://codereview.chromium.org/195993005/diff/1/pkg/barback/test/package_gra...
> pkg/barback/test/package_graph/collisions_test.dart:15: test("errors if two
> transformers output the same file", () {
> On 2014/03/13 16:56:47, Bob Nystrom wrote:
> > Why are these tests deleted?
> 
> This is a new file, collisions_test; these tests stayed in the old file,
> errors_test. I split them out because errors_test was getting pretty big.
> 
> Now that I look at it a couple of these still have to do with collisions,
> though, so I'll move them in here.

Oh, sorry. I didn't see that collisions_test was added.

LGTM!

[nweiz, 2014-03-13 20:14:22]

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