Removed supressions of test failures on Safari

Removed supressions of test failures on Safari

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

[ngeoffray, 2012-12-11 11:55:19]

LGTM! Thanks Martin

[ricow1, 2012-12-11 12:01:05]

https://codereview.chromium.org/11535003/diff/1/tests/html/html.status
File tests/html/html.status (right):

https://codereview.chromium.org/11535003/diff/1/tests/html/html.status#newcod...
tests/html/html.status:168: # The following tests all fail in Safari 5.1. (NOT
Mountain Lion). These will go
This comment is now outdated, the tests below do not have a valid reason for
failing

[Emily Fortuna, 2012-12-11 17:54:10]

https://codereview.chromium.org/11535003/diff/1/tests/html/html.status
File tests/html/html.status (right):

https://codereview.chromium.org/11535003/diff/1/tests/html/html.status#newcod...
tests/html/html.status:168: # The following tests all fail in Safari 5.1. (NOT
Mountain Lion). These will go
On 2012/12/11 12:01:05, ricow1 wrote:
> This comment is now outdated, the tests below do not have a valid reason for
> failing

+1 Please remove this comment, Martin.

[ricow1, 2012-12-11 17:57:24]

https://codereview.chromium.org/11535003/diff/1/tests/html/html.status
File tests/html/html.status (right):

https://codereview.chromium.org/11535003/diff/1/tests/html/html.status#newcod...
tests/html/html.status:168: # The following tests all fail in Safari 5.1. (NOT
Mountain Lion). These will go
On 2012/12/11 17:54:10, Emily Fortuna wrote:
> On 2012/12/11 12:01:05, ricow1 wrote:
> > This comment is now outdated, the tests below do not have a valid reason for
> > failing
> 
> +1 Please remove this comment, Martin.

Well, we correctly need to triage these, so either put in a comment stating that
this needs to be triaged using the standard format or have one of the dart2js
guys do it  while removing these

[Emily Fortuna, 2012-12-11 18:00:09]

https://codereview.chromium.org/11535003/diff/1/tests/html/html.status
File tests/html/html.status (right):

https://codereview.chromium.org/11535003/diff/1/tests/html/html.status#newcod...
tests/html/html.status:168: # The following tests all fail in Safari 5.1. (NOT
Mountain Lion). These will go
On 2012/12/11 17:57:24, ricow1 wrote:
> On 2012/12/11 17:54:10, Emily Fortuna wrote:
> > On 2012/12/11 12:01:05, ricow1 wrote:
> > > This comment is now outdated, the tests below do not have a valid reason
for
> > > failing
> > 
> > +1 Please remove this comment, Martin.
> 
> Well, we correctly need to triage these, so either put in a comment stating
that
> this needs to be triaged using the standard format or have one of the dart2js
> guys do it  while removing these

I agree; it's just the comment is not relevant now, so we should put a different
comment explaining what needs to be done.