Remove/skip invalid mirror tests.

Remove/skip invalid mirror tests.

-class of 1 is not required to declare int as an interface
-owner of members of mixin applications is the mixin
-MirrorsUsed does not have a semantics

R=gbracha@google.com

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

[rmacnak, 2014-11-06 00:31:27]

rmacnak@google.com changed reviewers:
+ gbracha@google.com

[gbracha, 2014-11-06 00:51:17]

I am ok with these changes (one minor comment) but you should be sure that
Peter/dart2js are ok with dropping the MirrorsUsed stuff?

https://codereview.chromium.org/706793002/diff/1/tests/lib/mirrors/intercepte...
File tests/lib/mirrors/intercepted_object_test.dart (right):

https://codereview.chromium.org/706793002/diff/1/tests/lib/mirrors/intercepte...
tests/lib/mirrors/intercepted_object_test.dart:32: }
This just adds the reflexive case, i.e. class C implements C, right?
A comment would not be amiss.

[rmacnak, 2014-11-06 01:02:44]

On 2014/11/06 00:51:17, gbracha wrote:
> I am ok with these changes (one minor comment) but you should be sure that
> Peter/dart2js are ok with dropping the MirrorsUsed stuff?

This only skips them on the VM.  (Although I would like to see them deleted.)

[rmacnak, 2014-11-06 01:02:49]

https://codereview.chromium.org/706793002/diff/1/tests/lib/mirrors/intercepte...
File tests/lib/mirrors/intercepted_object_test.dart (right):

https://codereview.chromium.org/706793002/diff/1/tests/lib/mirrors/intercepte...
tests/lib/mirrors/intercepted_object_test.dart:32: }
On 2014/11/06 00:51:17, gbracha wrote:
> This just adds the reflexive case, i.e. class C implements C, right?
> A comment would not be amiss.

Added.

[gbracha, 2014-11-06 01:08:48]

lgtm

[rmacnak, 2014-11-06 01:10:41]

Committed patchset #2 (id:10001) manually as 41540 (presubmit successful).

[floitsch, 2014-11-06 12:54:41]

floitsch@google.com changed reviewers:
+ floitsch@google.com

[floitsch, 2014-11-06 12:54:42]

Why did you remove the typedef_declaration_test?
It is testing something for dart2js.

[rmacnak, 2014-11-06 17:51:04]

On 2014/11/06 12:54:42, floitsch wrote:
> Why did you remove the typedef_declaration_test?
> It is testing something for dart2js.

It was renamed to mirrors_used_typedef_declaration_test, not removed.

[floitsch, 2014-11-07 10:03:24]

On 2014/11/06 17:51:04, rmacnak wrote:
> On 2014/11/06 12:54:42, floitsch wrote:
> > Why did you remove the typedef_declaration_test?
> > It is testing something for dart2js.
> 
> It was renamed to mirrors_used_typedef_declaration_test, not removed.

Thanks! I don't know how I missed that.