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

Unified Diff: trunk/src/net/http/transport_security_state.cc

Issue 24251011: Revert 224269 "Don't persist HPKP if PrivacyMode is enabled." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 3 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: trunk/src/net/http/transport_security_state.cc
===================================================================
--- trunk/src/net/http/transport_security_state.cc (revision 224274)
+++ trunk/src/net/http/transport_security_state.cc (working copy)
@@ -137,7 +137,6 @@
bool TransportSecurityState::GetDomainState(const std::string& host,
bool sni_enabled,
- bool allow_dynamic,
DomainState* result) {
DCHECK(CalledOnValidThread());
@@ -148,12 +147,6 @@
bool has_preload = GetStaticDomainState(canonicalized_host, sni_enabled,
&state);
- // If |allow_dynamic| is false, then return static state to the caller.
- if (!allow_dynamic) {
- if (has_preload)
- *result = state;
- return has_preload;
- }
std::string canonicalized_preload = CanonicalizeHost(state.domain);
GetDynamicDomainState(host, &state);
@@ -843,7 +836,8 @@
: upgrade_mode(MODE_DEFAULT),
created(base::Time::Now()),
sts_include_subdomains(false),
- pkp_include_subdomains(false) {}
+ pkp_include_subdomains(false) {
+}
TransportSecurityState::DomainState::~DomainState() {
}
« no previous file with comments | « trunk/src/net/http/transport_security_state.h ('k') | trunk/src/net/http/transport_security_state_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698