fix linter on svg

fix linter on svg

R=sigmund@google.com

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

[Jennifer Messerly, 2013-12-12 20:17:18]

XML namespaces are such fun :)

[Siggi Cherem (dart-lang), 2013-12-12 20:23:19]

lgtm, thanks :)

https://codereview.chromium.org/114473002/diff/1/pkg/polymer/test/build/linte...
File pkg/polymer/test/build/linter_test.dart (right):

https://codereview.chromium.org/114473002/diff/1/pkg/polymer/test/build/linte...
pkg/polymer/test/build/linter_test.dart:509: }, {});
This nuance will go away in my cl (will submit shortly), but note that this
should be 
{
  'a|lib/test.html.messages': ''
}


The problem here is that we check the output for what's listed, if no .messages
file are listed, we are not checking that the output is actually empty. This
however still detects that we don't crash, so for the purpose of this fix, this
is fine.

[Jennifer Messerly, 2013-12-12 20:42:01]

https://codereview.chromium.org/114473002/diff/1/pkg/polymer/test/build/linte...
File pkg/polymer/test/build/linter_test.dart (right):

https://codereview.chromium.org/114473002/diff/1/pkg/polymer/test/build/linte...
pkg/polymer/test/build/linter_test.dart:509: }, {});
On 2013/12/12 20:23:19, Siggi Cherem (dart-lang) wrote:
> This nuance will go away in my cl (will submit shortly), but note that this
> should be 
> {
>   'a|lib/test.html.messages': ''
> }
> 
> 
> The problem here is that we check the output for what's listed, if no
.messages
> file are listed, we are not checking that the output is actually empty. This
> however still detects that we don't crash, so for the purpose of this fix,
this
> is fine.

right. the test was definitely failing before I fixed it. :)
since you're about to submit I guess I'll just leave this.

[Jennifer Messerly, 2013-12-12 20:42:17]

Committed patchset #1 manually as r31108 (presubmit successful).