Add _Bigint._mulAdd to the invisible list. Strengthen the mirror invocation fuzzer.

Add _Bigint._mulAdd to the invisible list. Strengthen the mirror invocation fuzzer.

R=regis@google.com

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

[rmacnak, 2014-09-19 18:25:00]

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

[regis, 2014-09-22 16:20:43]

LGTM

https://codereview.chromium.org/590513003/diff/20001/tests/lib/mirrors/invoca...
File tests/lib/mirrors/invocation_fuzz_test.dart (right):

https://codereview.chromium.org/590513003/diff/20001/tests/lib/mirrors/invoca...
tests/lib/mirrors/invocation_fuzz_test.dart:6: // invoke them will all nulls.
This may result in Dart exceptions or hangs, but
It's not just all nulls anymore.

https://codereview.chromium.org/590513003/diff/20001/tests/lib/mirrors/invoca...
tests/lib/mirrors/invocation_fuzz_test.dart:145: //print(task.name);
Remove commented out line?

https://codereview.chromium.org/590513003/diff/20001/tests/lib/mirrors/invoca...
tests/lib/mirrors/invocation_fuzz_test.dart:164: fuzzArgument = new List(0); 
/// emptyarray: ok
Do I understand correctly that we can now name multi tests? Neat.

[rmacnak, 2014-09-22 22:42:12]

Also marked dart2js as failing when minified

https://codereview.chromium.org/590513003/diff/20001/tests/lib/mirrors/invoca...
File tests/lib/mirrors/invocation_fuzz_test.dart (right):

https://codereview.chromium.org/590513003/diff/20001/tests/lib/mirrors/invoca...
tests/lib/mirrors/invocation_fuzz_test.dart:6: // invoke them will all nulls.
This may result in Dart exceptions or hangs, but
On 2014/09/22 16:20:43, regis wrote:
> It's not just all nulls anymore.

Updated

https://codereview.chromium.org/590513003/diff/20001/tests/lib/mirrors/invoca...
tests/lib/mirrors/invocation_fuzz_test.dart:145: //print(task.name);
On 2014/09/22 16:20:43, regis wrote:
> Remove commented out line?

Uncommenting so this is on for the bot.

https://codereview.chromium.org/590513003/diff/20001/tests/lib/mirrors/invoca...
tests/lib/mirrors/invocation_fuzz_test.dart:164: fuzzArgument = new List(0); 
/// emptyarray: ok
On 2014/09/22 16:20:43, regis wrote:
> Do I understand correctly that we can now name multi tests? Neat.

Yep

[rmacnak, 2014-09-22 22:46:48]

Committed patchset #3 (id:40001) manually as 40576 (presubmit successful).