From dcheng:

This is a nit, but I think these conditions would be a bit easier to parse if they were phrased affirmatively instead of negatively, i.e.

bool IsValidFilter();
bool IsValidRequestScanOptions();
bool IsValidRequestDeviceOptions();

Then below, we would write

return !filters->empty() && std::all_of(filters->begin(), filters->end(), IsValidFilter);

Which I think will remain a bit more understandable in the future.