### Preflight Checklist

I have read the [Contributing Guidelines](https://github.com/electron/electron/blob/master/CONTRIBUTING.md) for this project.
I agree to follow the [Code of Conduct](https://github.com/electron/electron/blob/master/CODE_OF_CONDUCT.md) that this project adheres to.
I have searched the [issue tracker](https://www.github.com/electron/electron/issues) for a feature request that matches the one I want to file, without success.

### Electron Version

12.0.2

### What operating system are you using?

macOS

### Operating System Version

macOS Catalina 10.15.7

### What arch are you using?

x64

### Last Known Working Electron version

_No response_

### Expected Behavior

Returning an invalid serial port ID in the callback of the `select-serial-port` event handler of `session` should not crash Electron.

### Actual Behavior

Returning an invalid serial port ID in the callback of the `select-serial-port` event handler of `session` crashes Electron with a SIGSEGV. Calling the callback with `"foo"` or any string that is clearly not a valid serial port ID is enough to trigger the bug. Valid serial port IDs and empty strings work as expected.

### Testcase Gist URL

_No response_


shell/browser/serial/serial_chooser_controller.cc
const device::mojom::SerialPortInfo& port) const {
  if (filters_.empty())
    return true;
  for (const auto& filter : filters_) {
    if (filter->has_vendor_id &&
        (!port.has_vendor_id || filter->vendor_id != port.vendor_id)) {
      continue;
    }
    if (filter->has_product_id &&
        (!port.has_product_id || filter->product_id != port.product_id)) {
      continue;
    }
    return true;
  }
  return false;
}
void SerialChooserController::RunCallback(
    device::mojom::SerialPortInfoPtr port) {
  if (callback_) {
    std::move(callback_).Run(std::move(port));
  }
}
}