Replace PrefService::GetValueDict and PrefService::GetValueList
Semi-final step go/trust-your-prefs#heading=h.jjuz6fybcx8y
Introduce
- PrefService::GetDict and
- PrefService::GetList
as drop-in replacements for
- PrefService::GetValueDict and
- PrefService::GetValueList
and replace all known call sites via
sed -i 's/\bGetValueDict\b/GetDict/g' $(ag -l -s '\bGetValueDict\b')
sed -i 's/\bGetValueList\b/GetList/g' $(ag -l -s '\bGetValueList\b')
The final step will be to remove GetValueDict and GetValueList in a
follow-up CL.
Bug: 1334665
Test: Existing unit tests
Change-Id: I093e81af89628a02216501e59c1de7f1e5e22b9f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3887802
Auto-Submit: Roland Bock <[email protected]>
Reviewed-by: Gabriel Charette <[email protected]>
Owners-Override: Gabriel Charette <[email protected]>
Commit-Queue: Gabriel Charette <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1045895}
diff --git a/chrome/browser/renderer_preferences_util.cc b/chrome/browser/renderer_preferences_util.cc
index 42f1db8..6ed0a00 100644
--- a/chrome/browser/renderer_preferences_util.cc
+++ b/chrome/browser/renderer_preferences_util.cc
@@ -137,7 +137,7 @@
&prefs->webrtc_udp_max_port);
const base::Value::List& allowed_urls =
- pref_service->GetValueList(prefs::kWebRtcLocalIpsAllowedUrls);
+ pref_service->GetList(prefs::kWebRtcLocalIpsAllowedUrls);
prefs->webrtc_local_ips_allowed_urls = GetLocalIpsAllowedUrls(allowed_urls);
prefs->webrtc_allow_legacy_tls_protocols =
pref_service->GetBoolean(prefs::kWebRTCAllowLegacyTLSProtocols);