Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1881)

Unified Diff: chrome/browser/chromeos/policy/network_configuration_updater_impl.cc

Issue 20041002: Make CertificateHandler a proper interface of CertificateImporter. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments. Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/policy/network_configuration_updater_impl.cc
diff --git a/chrome/browser/chromeos/policy/network_configuration_updater_impl.cc b/chrome/browser/chromeos/policy/network_configuration_updater_impl.cc
index 262e77614301e6e4b898c90abeeb2ce08387fc96..de3a821a1ea240961f9a505b843c9cb63c647dfc 100644
--- a/chrome/browser/chromeos/policy/network_configuration_updater_impl.cc
+++ b/chrome/browser/chromeos/policy/network_configuration_updater_impl.cc
@@ -12,8 +12,8 @@
#include "base/values.h"
#include "chrome/browser/chromeos/net/onc_utils.h"
#include "chrome/browser/policy/policy_map.h"
-#include "chromeos/network/certificate_handler.h"
#include "chromeos/network/managed_network_configuration_handler.h"
+#include "chromeos/network/onc/onc_certificate_importer.h"
#include "chromeos/network/onc/onc_constants.h"
#include "chromeos/network/onc/onc_utils.h"
#include "policy/policy_constants.h"
@@ -22,13 +22,13 @@ namespace policy {
NetworkConfigurationUpdaterImpl::NetworkConfigurationUpdaterImpl(
PolicyService* device_policy_service,
- scoped_ptr<chromeos::CertificateHandler> certificate_handler)
+ scoped_ptr<chromeos::onc::CertificateImporter> certificate_importer)
: device_policy_change_registrar_(device_policy_service,
PolicyNamespace(POLICY_DOMAIN_CHROME,
std::string())),
user_policy_service_(NULL),
device_policy_service_(device_policy_service),
- certificate_handler_(certificate_handler.Pass()) {
+ certificate_importer_(certificate_importer.Pass()) {
device_policy_change_registrar_.Observe(
key::kDeviceOpenNetworkConfiguration,
base::Bind(&NetworkConfigurationUpdaterImpl::OnPolicyChanged,
@@ -156,7 +156,7 @@ void NetworkConfigurationUpdaterImpl::ApplyNetworkConfiguration(
onc_blob, onc_source, "", &network_configs, &certificates);
scoped_ptr<net::CertificateList> web_trust_certs(new net::CertificateList);
- certificate_handler_->ImportCertificates(
+ certificate_importer_->ImportCertificates(
certificates, onc_source, web_trust_certs.get());
std::string userhash;

Powered by Google App Engine
This is Rietveld 408576698