Quick style question (also applies below): what do you feel about making the "exceptional case" (`region == null`, I'm guessing?) the bit you handle in the `if...` clause, e.g.
```
if (region == null) {
  return ImmutableSet.of(); // handle special case
}
return listSecurityGroupsInLocation(region);
```
?