Allow infering of return types on runtime helpers. It's only arugments the inferrer does not see.

Allow infering of return types on runtime helpers. It's only arugments the inferrer does not see.
Also fix a few dependencies of runtime errors requiring the string class to be instantiated.

R=ahe@google.com, karlklose@google.com

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

[ahe, 2013-08-15 11:27:43]

Nice! I think this is going to work really well for mirrors as well :-)

LGTM

https://codereview.chromium.org/22833005/diff/11001/sdk/lib/_internal/compile...
File sdk/lib/_internal/compiler/implementation/js_backend/backend.dart (right):

https://codereview.chromium.org/22833005/diff/11001/sdk/lib/_internal/compile...
sdk/lib/_internal/compiler/implementation/js_backend/backend.dart:906:
compiler.enqueuer.resolution.registerInstantiatedClass(
Please document that these represent the arguments passed to this method.

https://codereview.chromium.org/22833005/diff/11001/sdk/lib/_internal/compile...
sdk/lib/_internal/compiler/implementation/js_backend/backend.dart:912:
enqueuer.registerInstantiatedClass(compiler.stringClass, elements);
Ditto.

https://codereview.chromium.org/22833005/diff/11001/sdk/lib/_internal/compile...
sdk/lib/_internal/compiler/implementation/js_backend/backend.dart:917:
enqueuer.registerInstantiatedClass(compiler.stringClass, elements);
Ditto.

[ngeoffray, 2013-08-15 11:31:34]

Thanks Peter.

https://codereview.chromium.org/22833005/diff/11001/sdk/lib/_internal/compile...
File sdk/lib/_internal/compiler/implementation/js_backend/backend.dart (right):

https://codereview.chromium.org/22833005/diff/11001/sdk/lib/_internal/compile...
sdk/lib/_internal/compiler/implementation/js_backend/backend.dart:906:
compiler.enqueuer.resolution.registerInstantiatedClass(
On 2013/08/15 11:27:43, ahe wrote:
> Please document that these represent the arguments passed to this method.

Done.

https://codereview.chromium.org/22833005/diff/11001/sdk/lib/_internal/compile...
sdk/lib/_internal/compiler/implementation/js_backend/backend.dart:912:
enqueuer.registerInstantiatedClass(compiler.stringClass, elements);
On 2013/08/15 11:27:43, ahe wrote:
> Ditto.

Done.

https://codereview.chromium.org/22833005/diff/11001/sdk/lib/_internal/compile...
sdk/lib/_internal/compiler/implementation/js_backend/backend.dart:917:
enqueuer.registerInstantiatedClass(compiler.stringClass, elements);
On 2013/08/15 11:27:43, ahe wrote:
> Ditto.

Done.

[karlklose, 2013-08-15 11:32:54]

LGTM.

[ngeoffray, 2013-08-15 11:53:49]

Committed patchset #5 manually as r26189 (presubmit successful).