Interstitial options are not OR'd properly.

content/browser/ssl/ssl_policy.h

 // Copyright (c) 2012 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 CONTENT_BROWSER_SSL_SSL_POLICY_H_
 #define CONTENT_BROWSER_SSL_SSL_POLICY_H_
 
 #include <string>
 
 #include "base/memory/ref_counted.h"
@@ skipping 26 matching lines @@
   void OnRequestStarted(SSLRequestInfo* info);
 
   // Update the SSL information in |entry| to match the current state.
   // |web_contents| is the WebContentsImpl associated with this entry.
   void UpdateEntry(NavigationEntryImpl* entry,
                    WebContentsImpl* web_contents);
 
   SSLPolicyBackend* backend() const { return backend_; }
 
  private:
-  enum OnCertErrorInternalOptionsMask {
-    OVERRIDABLE = 1 << 0,
-    STRICT_ENFORCEMENT = 1 << 1,
-    EXPIRED_PREVIOUS_DECISION = 1 << 2
-  };
-
   // Callback that the user chose to accept or deny the certificate.
   void OnAllowCertificate(scoped_refptr<SSLCertErrorHandler> handler,
                           bool allow);
 
   // Helper method for derived classes handling certificate errors.
   //
-  // Options should be a bitmask combination of OnCertErrorInternalOptionsMask.
-  // OVERRIDABLE indicates whether or not the user could (assuming perfect
-  // knowledge) successfully override the error and still get the security
-  // guarantees of TLS. STRICT_ENFORCEMENT indicates whether or not the site the
-  // user is trying to connect to has requested strict enforcement of
-  // certificate validation (e.g. with HTTP Strict-Transport-Security).
-  // EXPIRED_PREVIOUS_DECISION indicates whether a user decision had been
-  // previously made but the decision has expired.
+  // |options_mask| should be a bitmask combination of
+  // ContentBrowserClient::AllowCertErrorOptionsMask. OVERRIDABLE indicates
+  // whether or not the user could (assuming perfect knowledge) successfully
+  // override the error and still get the security guarantees of TLS.
+  // STRICT_ENFORCEMENT indicates whether or not the site the user is trying to
+  // connect to has requested strict enforcement of certificate validation (e.g.
+  // with HTTP Strict-Transport-Security). EXPIRED_PREVIOUS_DECISION indicates
+  // whether a user decision had been previously made but the decision has
+  // expired.
   void OnCertErrorInternal(SSLCertErrorHandler* handler, int options_mask);
 
   // If the security style of |entry| has not been initialized, then initialize
   // it with the default style for its URL.
   void InitializeEntryIfNeeded(NavigationEntryImpl* entry);
 
   // Mark |origin| as having run insecure content in the process with ID |pid|.
   void OriginRanInsecureContent(const std::string& origin, int pid);
 
   // The backend we use to enact our decisions.
   SSLPolicyBackend* backend_;
 
   DISALLOW_COPY_AND_ASSIGN(SSLPolicy);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_SSL_SSL_POLICY_H_