diff --git a/src/net/url_request/url_request_http_job.cc b/src/net/url_request/url_request_http_job.cc index b8c0f6ba..af07aca9 100644 --- a/src/net/url_request/url_request_http_job.cc +++ b/src/net/url_request/url_request_http_job.cc @@ -108,18 +108,18 @@ base::Value CookieInclusionStatusNetLogParams( const std::string& cookie_path, const net::CookieInclusionStatus& status, net::NetLogCaptureMode capture_mode) { - base::Value dict(base::Value::Type::DICTIONARY); - dict.SetStringKey("operation", operation); - dict.SetStringKey("status", status.GetDebugString()); + base::Value::Dict dict; + dict.Set("operation", operation); + dict.Set("status", status.GetDebugString()); if (net::NetLogCaptureIncludesSensitive(capture_mode)) { if (!cookie_name.empty()) - dict.SetStringKey("name", cookie_name); + dict.Set("name", cookie_name); if (!cookie_domain.empty()) - dict.SetStringKey("domain", cookie_domain); + dict.Set("domain", cookie_domain); if (!cookie_path.empty()) - dict.SetStringKey("path", cookie_path); + dict.Set("path", cookie_path); } - return dict; + return base::Value(std::move(dict)); } // Records details about the most-specific trust anchor in |spki_hashes|, @@ -623,10 +623,6 @@ void URLRequestHttpJob::AddCookieHeaderAndStart() { same_site_context, first_party_set_metadata_.context(), request_->isolation_info(), is_in_nontrivial_first_party_set); - UMA_HISTOGRAM_ENUMERATION( - "Cookie.FirstPartySetsContextType.HTTP.Read", - first_party_set_metadata_.first_party_sets_context_type()); - cookie_store->GetCookieListWithOptionsAsync( request_->url(), options, CookiePartitionKeyCollection::FromOptional(cookie_partition_key_.value()), @@ -863,10 +859,6 @@ void URLRequestHttpJob::SaveCookiesAndNotifyHeadersComplete(int result) { same_site_context, first_party_set_metadata_.context(), request_->isolation_info(), is_in_nontrivial_first_party_set); - UMA_HISTOGRAM_ENUMERATION( - "Cookie.FirstPartySetsContextType.HTTP.Write", - first_party_set_metadata_.first_party_sets_context_type()); - // Set all cookies, without waiting for them to be set. Any subsequent // read will see the combined result of all cookie operation. const base::StringPiece name("Set-Cookie");