summaryrefslogtreecommitdiffstats
path: root/src/core/net
diff options
context:
space:
mode:
authorMichal Klocek <michal.klocek@qt.io>2022-11-09 17:34:06 +0100
committerMichal Klocek <michal.klocek@qt.io>2022-11-22 08:26:09 +0100
commitc52afb2206ccfa46b1a09ef7af2cb0ae210e3b79 (patch)
tree574dc1ef28e32a05e429037d333e347d90eebb12 /src/core/net
parentdaaac7adb519e82b21a2f826ef6ae83c4f102a62 (diff)
Minor. Remove Override from client certificate store
Change 'Override" in favor of 'Qt' so it matches the rest of the source base. Pick-to: 6.4 Change-Id: I0ba614ed3017a8d7e0ba5fc703b3f04c28b5ead7 Reviewed-by: Michael BrĂ¼ning <michael.bruning@qt.io>
Diffstat (limited to 'src/core/net')
-rw-r--r--src/core/net/client_cert_qt.cpp (renamed from src/core/net/client_cert_override.cpp)28
-rw-r--r--src/core/net/client_cert_qt.h (renamed from src/core/net/client_cert_override.h)10
-rw-r--r--src/core/net/client_cert_store_data.cpp8
3 files changed, 23 insertions, 23 deletions
diff --git a/src/core/net/client_cert_override.cpp b/src/core/net/client_cert_qt.cpp
index d1946de5d..1e7aa8b6a 100644
--- a/src/core/net/client_cert_override.cpp
+++ b/src/core/net/client_cert_qt.cpp
@@ -1,7 +1,7 @@
// Copyright (C) 2018 The Qt Company Ltd.
// SPDX-License-Identifier: LicenseRef-Qt-Commercial OR LGPL-3.0-only OR GPL-2.0-only OR GPL-3.0-only
-#include "client_cert_override.h"
+#include "client_cert_qt.h"
#include "base/bind.h"
#include "base/task/post_task.h"
@@ -35,12 +35,12 @@
namespace {
-class ClientCertIdentityOverride : public net::ClientCertIdentity
+class ClientCertIdentityQt : public net::ClientCertIdentity
{
public:
- ClientCertIdentityOverride(scoped_refptr<net::X509Certificate> cert, scoped_refptr<net::SSLPrivateKey> key)
+ ClientCertIdentityQt(scoped_refptr<net::X509Certificate> cert, scoped_refptr<net::SSLPrivateKey> key)
: net::ClientCertIdentity(std::move(cert)), m_key(std::move(key)) {}
- ~ClientCertIdentityOverride() override = default;
+ ~ClientCertIdentityQt() override = default;
void AcquirePrivateKey(base::OnceCallback<void(scoped_refptr<net::SSLPrivateKey>)> private_key_callback) override
{
@@ -55,17 +55,17 @@ private:
namespace QtWebEngineCore {
-ClientCertOverrideStore::ClientCertOverrideStore(ClientCertificateStoreData *storeData)
+ClientCertStoreQt::ClientCertStoreQt(ClientCertificateStoreData *storeData)
: ClientCertStore()
, m_storeData(storeData)
, m_nativeStore(createNativeStore())
{
}
-ClientCertOverrideStore::~ClientCertOverrideStore() = default;
+ClientCertStoreQt::~ClientCertStoreQt() = default;
#if QT_CONFIG(ssl)
-net::ClientCertIdentityList ClientCertOverrideStore::GetClientCertsOnUIThread(const net::SSLCertRequestInfo &cert_request_info)
+net::ClientCertIdentityList ClientCertStoreQt::GetClientCertsOnUIThread(const net::SSLCertRequestInfo &cert_request_info)
{
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
const auto &clientCertOverrideData = m_storeData->extraCerts;
@@ -81,7 +81,7 @@ net::ClientCertIdentityList ClientCertOverrideStore::GetClientCertsOnUIThread(co
}
if (cert_request_info.cert_authorities.empty()
|| cert->IsIssuedByEncoded(cert_request_info.cert_authorities)) {
- selected_identities.push_back(std::make_unique<ClientCertIdentityOverride>(
+ selected_identities.push_back(std::make_unique<ClientCertIdentityQt>(
cert, clientCertOverrideData[i]->keyPtr));
}
}
@@ -89,14 +89,14 @@ net::ClientCertIdentityList ClientCertOverrideStore::GetClientCertsOnUIThread(co
return selected_identities;
}
-void ClientCertOverrideStore::GetClientCertsReturn(const net::SSLCertRequestInfo &cert_request_info,
+void ClientCertStoreQt::GetClientCertsReturn(const net::SSLCertRequestInfo &cert_request_info,
ClientCertListCallback callback,
net::ClientCertIdentityList &&result)
{
// Continue with native cert store and append them after memory certificates
if (m_nativeStore) {
ClientCertListCallback callback2 = base::BindOnce(
- [](ClientCertOverrideStore::ClientCertListCallback callback,
+ [](ClientCertStoreQt::ClientCertListCallback callback,
net::ClientCertIdentityList result1, net::ClientCertIdentityList result2) {
while (!result2.empty()) {
result1.push_back(std::move(result2.back()));
@@ -113,16 +113,16 @@ void ClientCertOverrideStore::GetClientCertsReturn(const net::SSLCertRequestInfo
#endif // QT_CONFIG(ssl)
-void ClientCertOverrideStore::GetClientCerts(const net::SSLCertRequestInfo &cert_request_info,
+void ClientCertStoreQt::GetClientCerts(const net::SSLCertRequestInfo &cert_request_info,
ClientCertListCallback callback)
{
#if QT_CONFIG(ssl)
// Access the user-provided data from the UI thread, but return on whatever thread this is.
bool ok = base::PostTaskAndReplyWithResult(
FROM_HERE, { content::BrowserThread::UI },
- base::BindOnce(&ClientCertOverrideStore::GetClientCertsOnUIThread,
+ base::BindOnce(&ClientCertStoreQt::GetClientCertsOnUIThread,
base::Unretained(this), std::cref(cert_request_info)),
- base::BindOnce(&ClientCertOverrideStore::GetClientCertsReturn,
+ base::BindOnce(&ClientCertStoreQt::GetClientCertsReturn,
base::Unretained(this), std::cref(cert_request_info), std::move(callback)));
DCHECK(ok); // callback is already moved and we can't really recover here.
#else
@@ -134,7 +134,7 @@ void ClientCertOverrideStore::GetClientCerts(const net::SSLCertRequestInfo &cert
}
// static
-std::unique_ptr<net::ClientCertStore> ClientCertOverrideStore::createNativeStore()
+std::unique_ptr<net::ClientCertStore> ClientCertStoreQt::createNativeStore()
{
#if BUILDFLAG(USE_NSS_CERTS)
return std::unique_ptr<net::ClientCertStore>(new net::ClientCertStoreNSS(net::ClientCertStoreNSS::PasswordDelegateFactory()));
diff --git a/src/core/net/client_cert_override.h b/src/core/net/client_cert_qt.h
index 6f740cc9c..85b0d28be 100644
--- a/src/core/net/client_cert_override.h
+++ b/src/core/net/client_cert_qt.h
@@ -1,8 +1,8 @@
// Copyright (C) 2018 The Qt Company Ltd.
// SPDX-License-Identifier: LicenseRef-Qt-Commercial OR LGPL-3.0-only OR GPL-2.0-only OR GPL-3.0-only
-#ifndef CLIENT_CERT_OVERRIDE_P_H
-#define CLIENT_CERT_OVERRIDE_P_H
+#ifndef CLIENT_CERT_QT_P_H
+#define CLIENT_CERT_QT_P_H
#include "net/ssl/client_cert_store.h"
#include "base/callback_forward.h"
@@ -15,11 +15,11 @@ class SSLCertRequestInfo;
namespace QtWebEngineCore {
struct ClientCertificateStoreData;
-class ClientCertOverrideStore : public net::ClientCertStore
+class ClientCertStoreQt : public net::ClientCertStore
{
public:
- ClientCertOverrideStore(ClientCertificateStoreData *storeData);
- virtual ~ClientCertOverrideStore() override;
+ ClientCertStoreQt(ClientCertificateStoreData *storeData);
+ virtual ~ClientCertStoreQt() override;
void GetClientCerts(const net::SSLCertRequestInfo &cert_request_info,
ClientCertListCallback callback) override;
private:
diff --git a/src/core/net/client_cert_store_data.cpp b/src/core/net/client_cert_store_data.cpp
index 33ef6b5e9..306e782e0 100644
--- a/src/core/net/client_cert_store_data.cpp
+++ b/src/core/net/client_cert_store_data.cpp
@@ -20,16 +20,16 @@
namespace {
-class SSLPlatformKeyOverride : public net::ThreadedSSLPrivateKey::Delegate
+class SSLPlatformKeyQt : public net::ThreadedSSLPrivateKey::Delegate
{
public:
- SSLPlatformKeyOverride(const QByteArray &sslKeyInBytes)
+ SSLPlatformKeyQt(const QByteArray &sslKeyInBytes)
{
m_mem = BIO_new_mem_buf(sslKeyInBytes, -1);
m_key = PEM_read_bio_PrivateKey(m_mem, nullptr, nullptr, nullptr);
}
- ~SSLPlatformKeyOverride() override
+ ~SSLPlatformKeyQt() override
{
if (m_key)
EVP_PKEY_free(m_key);
@@ -82,7 +82,7 @@ scoped_refptr<net::SSLPrivateKey> wrapOpenSSLPrivateKey(const QByteArray &sslKey
return nullptr;
return base::MakeRefCounted<net::ThreadedSSLPrivateKey>(
- std::make_unique<SSLPlatformKeyOverride>(sslKeyInBytes),
+ std::make_unique<SSLPlatformKeyQt>(sslKeyInBytes),
net::GetSSLPlatformKeyTaskRunner());
}