Change bidi_utils and bidi_formatter for new optional parameters.

Change bidi_utils and bidi_formatter for new optional parameters.
Committed: https://code.google.com/p/dart/source/detail?r=13793

[regis, 2012-10-17 20:56:02]

Nicolas,

Either something is wrong with this cl, or this dart2js issue is valid:
http://code.google.com/p/dart/issues/detail?id=6015

Thanks,
Regis

[regis, 2012-10-17 21:05:11]

I just notice that Nicolas is in vacation. Thanks Peter!

-- Regis

[regis, 2012-10-18 16:42:44]

Thank you Peter for fixing the related dart2js bug.

Adding Anton as a reviewer, since dartium may rely on this html formatting code
(not sure).

Cheers,
Regis

[ahe, 2012-10-18 16:44:31]

I think Emily and Alan are the best to review this code.

[Alan Knight, 2012-10-18 18:05:28]

LGTM

The only real question seems to be if the intent is for those to be optional
positional parameters or named ones. I think named works better for those, so
this looks good.

[regis, 2012-10-18 18:35:39]

On 2012/10/18 18:05:28, Alan Knight wrote:
> LGTM
> 
> The only real question seems to be if the intent is for those to be optional
> positional parameters or named ones. I think named works better for those, so
> this looks good.

Thanks all!

[Emily Fortuna, 2012-10-18 21:33:47]

lgtm