@Hixie

Search for _emptyWidgetList in basic.dart. We should presumably replace it with inline const <Widget>[] everywhere it's used too.

LGTM.

Test maybe?

We should presumably replace it with inline const [] everywhere it's used too.

Ok. I'll do that in a separate patch.

Test maybe?

I guess I could test it... I'm not sure it would be such a valuable test though.

We should presumably replace it with inline const [] everywhere it's used too.

Ok. I'll do that in a separate patch.

Turns out I did it in this patch because I'm such a great guy.

Test maybe?

I guess I could test it... I'm not sure it would be such a valuable test though.

Alright, I tested that all these classes can be constructed with only required arguments. Turned up a bug (and a doc opportunity), so I guess it was worthwhile. :)

:-)