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

Unified Diff: chrome/browser/component_updater/sth_set_component_installer.h

Issue 1853753003: Certificate Transparency: New component for obtaining fresh STHs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ready for review Created 4 years, 8 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/component_updater/sth_set_component_installer.h
diff --git a/chrome/browser/component_updater/sth_set_component_installer.h b/chrome/browser/component_updater/sth_set_component_installer.h
new file mode 100644
index 0000000000000000000000000000000000000000..e5e705ec402e635b98fd93c817d29de3ea726a55
--- /dev/null
+++ b/chrome/browser/component_updater/sth_set_component_installer.h
@@ -0,0 +1,89 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_COMPONENT_UPDATER_STH_SET_COMPONENT_INSTALLER_H_
+#define CHROME_BROWSER_COMPONENT_UPDATER_STH_SET_COMPONENT_INSTALLER_H_
+
+#include <stdint.h>
+#include <string>
+#include <vector>
+
+#include "base/files/file_path.h"
+#include "base/gtest_prod_util.h"
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/values.h"
+#include "components/component_updater/default_component_installer.h"
+#include "net/cert/sth_observer.h"
+
+namespace base {
+class FilePath;
+class Value;
+} // namespace base
+
+namespace component_updater {
+
+class ComponentUpdateService;
+
+// Component for receiving Signed Tree Heads updates for Certificate
+// Transparency logs recognized in Chrome.
+// The STHs are in JSON format.
+// To identify the log each STH belongs to, the name of the file is
+// hex-encoded Log ID of the log that produced this STH.
+//
+// Unlike other components, once STHs are observed they are not placed
+// in some global object. Instead, a proxy class provided in the C'tor
Sorin Jianu 2016/04/04 22:06:13 maybe we just say constructor.
Eran Messeri 2016/04/05 15:34:07 Done.
+// sends notifications for each of the new STHs.
+class STHSetComponentInstallerTraits : public ComponentInstallerTraits {
+ public:
+ // The |sth_distributor| will be notified each time a new STH is observed.
+ explicit STHSetComponentInstallerTraits(
+ scoped_ptr<net::ct::STHObserver> sth_observer);
+ ~STHSetComponentInstallerTraits() override;
+
+ private:
+ friend class STHSetComponentInstallerTest;
+
+ // The following methods override ComponentInstallerTraits.
Sorin Jianu 2016/04/04 22:06:13 Please remember to rebase this change, since new m
Eran Messeri 2016/04/05 15:34:06 Done.
+ bool CanAutoUpdate() const override;
+ bool OnCustomInstall(const base::DictionaryValue& manifest,
+ const base::FilePath& install_dir) override;
+ bool VerifyInstallation(const base::DictionaryValue& manifest,
+ const base::FilePath& install_dir) const override;
+ void ComponentReady(const base::Version& version,
+ const base::FilePath& install_dir,
+ scoped_ptr<base::DictionaryValue> manifest) override;
+ base::FilePath GetBaseDirectory() const override;
+ void GetHash(std::vector<uint8_t>* hash) const override;
+ std::string GetName() const override;
+
+ static base::FilePath GetInstalledPath(const base::FilePath& base);
+
+ // Reads and parses the on-disk json.
+ void LoadSTHsFromDisk(const base::FilePath& sths_file_path,
+ const base::Version& version);
+
+ // Handle successful parsing of JSON by distributing the new STH.
+ void OnJsonParseSuccess(std::string log_id,
+ scoped_ptr<base::Value> parsed_json);
+
+ // STH parsing failed - do nothing.
+ void OnJsonParseError(std::string log_id, const std::string& error);
+
+ scoped_ptr<net::ct::STHObserver> sth_observer_;
+
+ FRIEND_TEST_ALL_PREFIXES(STHSetComponentInstallerTest, CanLoadAllSTHs);
+ FRIEND_TEST_ALL_PREFIXES(STHSetComponentInstallerTest, VerifyInstallation);
+ FRIEND_TEST_ALL_PREFIXES(STHSetComponentInstallerTest, LoadSTHsFromDisk);
+
+ DISALLOW_COPY_AND_ASSIGN(STHSetComponentInstallerTraits);
+};
+
+void RegisterSTHSetComponent(ComponentUpdateService* cus,
+ const base::FilePath& user_data_dir,
+ scoped_ptr<net::ct::STHObserver> sth_observer);
+
+} // namespace component_updater
+
+#endif // CHROME_BROWSER_COMPONENT_UPDATER_STH_SET_COMPONENT_INSTALLER_H_

Powered by Google App Engine
This is Rietveld 408576698