| commit | 6fbeca093b227a66cb6bcc56694d25ca7e2aee4b | [log] [tgz] |
|---|---|---|
| author | Sorin Jianu <[email protected]> | Wed Oct 09 17:31:52 2024 |
| committer | Chromium LUCI CQ <[email protected]> | Wed Oct 09 17:31:52 2024 |
| tree | 15dd7f06ebe9d1fddc521fd7cbc7afe1ff528825 | |
| parent | 7318cf7f03d15fbf020afd6361d11aa3ff861883 [diff] [blame] |
components: default the user-defined empty ctor and dtor where possible I need an owner's approval please for changes under: components/location This is a mechanical change. There are a few instances (but not all) of empty user-defined constructors and destructors which can be defaulted in the /components code. This CL was uploaded by git cl split. [email protected] Bug: 371316188 Change-Id: Icd446edd168e12ec6b6b8cf59c9e8cc4d1c0ff88 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5918685 Auto-Submit: Sorin Jianu <[email protected]> Reviewed-by: Alvin Ji <[email protected]> Commit-Queue: Alvin Ji <[email protected]> Cr-Commit-Position: refs/heads/main@{#1366294}
diff --git a/components/location/android/location_settings_impl.cc b/components/location/android/location_settings_impl.cc index 8b7d7670..5cf7948 100644 --- a/components/location/android/location_settings_impl.cc +++ b/components/location/android/location_settings_impl.cc
@@ -16,9 +16,9 @@ using LocationSettingsDialogOutcomeCallback = LocationSettings::LocationSettingsDialogOutcomeCallback; -LocationSettingsImpl::LocationSettingsImpl() {} +LocationSettingsImpl::LocationSettingsImpl() = default; -LocationSettingsImpl::~LocationSettingsImpl() {} +LocationSettingsImpl::~LocationSettingsImpl() = default; bool LocationSettingsImpl::HasAndroidLocationPermission() { JNIEnv* env = AttachCurrentThread();