Use SizedBox.shrink(...) and SizedBox.expand(...) constructors appropriately

[domesticmouse]

Describe the rule you'd like to see implemented

Implement a lint rule that catches usage of the normal SizedBox(...) constructor
that should instead use the SizedBox.shrink(...) and SizedBox.expand(...)
constructors instead.

This is a follow on from #2049 (comment)

Examples

BAD:

Widget buildLogo() {
  return SizedBox(
    height: 0,
    width: 0,
    child:const MyLogo(),
  );
}

Widget buildLogo() {
  return SizedBox(
    height: double.infinity,
    width: double.infinity,
    child:const MyLogo(),
  );
}

GOOD:

Widget buildLogo() {
  return SizedBox.shrink(
    child:const MyLogo(),
  );
}

Widget buildLogo() {
  return SizedBox.expand(
    child:const MyLogo(),
  );
}

[domesticmouse]

@bwilkerson and @pq I'm happy to have this bug assigned to me to implement, if we decide this lint is worth having.

cc: @a14n

[pq]

SGTM!