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

Unified Diff: content/browser/ssl/ssl_cert_error_handler.h

Issue 9188019: Cleanup: Rename is_hsts_host to fatal. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 8 years, 11 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: content/browser/ssl/ssl_cert_error_handler.h
===================================================================
--- content/browser/ssl/ssl_cert_error_handler.h (revision 117302)
+++ content/browser/ssl/ssl_cert_error_handler.h (working copy)
@@ -21,14 +21,14 @@
net::URLRequest* request,
ResourceType::Type resource_type,
const net::SSLInfo& ssl_info,
- bool is_hsts_host);
+ bool fatal);
virtual SSLCertErrorHandler* AsSSLCertErrorHandler() OVERRIDE;
// These accessors are available on either thread
const net::SSLInfo& ssl_info() const { return ssl_info_; }
int cert_error() const { return cert_error_; }
- bool is_hsts_host() const { return is_hsts_host_; }
+ bool fatal() const { return fatal_; }
protected:
// SSLErrorHandler methods
@@ -41,7 +41,8 @@
// These read-only members may be accessed on any thread.
const net::SSLInfo ssl_info_;
const int cert_error_; // The error we represent.
- const bool is_hsts_host_; // true if the error is from an HSTS host.
+ const bool fatal_; // True if the error is from a host requiring
+ // certificate errors to be fatal.
DISALLOW_COPY_AND_ASSIGN(SSLCertErrorHandler);
};
« no previous file with comments | « content/browser/renderer_host/resource_dispatcher_host.h ('k') | content/browser/ssl/ssl_cert_error_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698