Ternary operator support for polymer_expressions

Ternary operator support for polymer_expressions

BUG=
R=jmesserly@google.com

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

[justinfagnani, 2014-01-27 18:16:10]

On 2014/01/27 18:02:11, justinfagnani wrote:

Fixes https://code.google.com/p/dart/issues/detail?id=12663

[Jennifer Messerly, 2014-01-28 00:03:29]

LGTM with a couple of lines added to the test :)

https://codereview.chromium.org/131623004/diff/1/pkg/polymer_expressions/test...
File pkg/polymer_expressions/test/eval_test.dart (right):

https://codereview.chromium.org/131623004/diff/1/pkg/polymer_expressions/test...
pkg/polymer_expressions/test/eval_test.dart:118: expectEval('false ? 1 : false ?
2 : 3', 3);
maybe add a test for `null` and some other kind of value (e.g. 42) in the
condition?

[justinfagnani, 2014-01-28 01:05:21]

thanks!

https://codereview.chromium.org/131623004/diff/1/pkg/polymer_expressions/test...
File pkg/polymer_expressions/test/eval_test.dart (right):

https://codereview.chromium.org/131623004/diff/1/pkg/polymer_expressions/test...
pkg/polymer_expressions/test/eval_test.dart:118: expectEval('false ? 1 : false ?
2 : 3', 3);
On 2014/01/28 00:03:29, John Messerly wrote:
> maybe add a test for `null` and some other kind of value (e.g. 42) in the
> condition?

Done.

[justinfagnani, 2014-01-28 01:47:39]

Committed patchset #1 manually as r32048 (presubmit successful).