Refactor components/cronet to use PrefService::GetValue*

This is part of a larger migration effort based on
go/trust-your-prefs.

Bug: 1342019, 1341998, 1341996
Test: Existing unit tests
Change-Id: I369b140d903d33d566d853b4369d32a6aec91d24
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3810279
Commit-Queue: Roland Bock <rbock@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1032439}

components/cronet/cronet_prefs_manager.cc

@@ -116,7 +116,7 @@ class PrefServiceAdapter : public net::HttpServerProperties::PrefDelegate {
 
   // PrefDelegate implementation.
   const base::Value* GetServerProperties() const override {
-    return pref_service_->Get(path_);
+    return &pref_service_->GetValue(path_);
   }
 
   void SetServerProperties(const base::Value& value,
@@ -183,9 +183,7 @@ class NetworkQualitiesPrefDelegateImpl
   base::Value::Dict GetDictionaryValue() override {
     DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
     UMA_HISTOGRAM_EXACT_LINEAR("NQE.Prefs.ReadCount", 1, 2);
-    return pref_service_->GetDictionary(kNetworkQualitiesPref)
-        ->GetDict()
-        .Clone();
+    return pref_service_->GetValueDict(kNetworkQualitiesPref).Clone();
   }
 
  private:

components/cronet/host_cache_persistence_manager.cc

@@ -56,8 +56,7 @@ void HostCachePersistenceManager::ReadFromDisk() {
     return;
 
   net_log_.BeginEvent(net::NetLogEventType::HOST_CACHE_PREF_READ);
-  const base::Value::List& pref_value =
-      pref_service_->GetList(pref_name_)->GetList();
+  const base::Value::List& pref_value = pref_service_->GetValueList(pref_name_);
   bool success = cache_->RestoreFromListValue(pref_value);
   net_log_.AddEntryWithBoolParams(net::NetLogEventType::HOST_CACHE_PREF_READ,
                                   net::NetLogEventPhase::END, "success",