Expand WebsiteSettings histograms for HTTP-bad

chrome/browser/ui/website_settings/website_settings.h

Index: chrome/browser/ui/website_settings/website_settings.h
diff --git a/chrome/browser/ui/website_settings/website_settings.h b/chrome/browser/ui/website_settings/website_settings.h
index f2606784f7e3ab360423194bcda40950bd31e035..4a64b3ab186b8de393d31166549315613603c548 100644
--- a/chrome/browser/ui/website_settings/website_settings.h
+++ b/chrome/browser/ui/website_settings/website_settings.h
@@ -236,6 +236,9 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver,
 
   Profile* profile_;
 
+  security_state::SecurityStateModel::SecurityLevel security_level_ =

[lgarron, 2016/10/21 01:04:57]

I don't know our C+ conventions perfectly, but shouldn't this get initialized
with al the other instance variables in the constructor?

[estark, 2016/10/21 01:16:47]

In-class member initializers are allowed by the style guide, see
https://chromium-cpp.appspot.com/ and
https://groups.google.com/a/chromium.org/d/msg/chromium-dev/zqB-DySA4V0/eS9wV...

An OWNER might come along and tell me to be consistent with the rest of the file
though, so we shall see.

[lgarron, 2016/10/21 01:22:50]

Alright, sounds fine to me. Thanks for the link!

+      security_state::SecurityStateModel::NONE;
+
   DISALLOW_COPY_AND_ASSIGN(WebsiteSettings);
 };