Add comment to explain why some of the strings are typed as string16, and a TODO as a reminder to c…

chrome/browser/website_settings.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 CHROME_BROWSER_WEBSITE_SETTINGS_H_
 #define CHROME_BROWSER_WEBSITE_SETTINGS_H_
 
 #include "base/string16.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/content_settings/tab_specific_content_settings.h"
@@ skipping 129 matching lines @@
   // The Omnibox URL of the website for which to display site permissions and
   // site information.
   GURL site_url_;
 
   // Status of the website's identity verification check.
   SiteIdentityStatus site_identity_status_;
 
   // Status of the connection to the website.
   SiteConnectionStatus site_connection_status_;
 
+  // TODO(markusheintz): Move the creation of all the string16 typed UI
+  // strings below to the corresponding UI code, in order to prevent
+  // unnesseccary UTF string conversions.

[wtc, 2012/04/23 21:49:30]

Typo: unnesseccary => unnecessary

Should this be "UTF-8 string conversions"?

[markusheintz_, 2012/04/24 08:52:59]

Done.

+
   // Details about the website's identity. If the website's identity has been
   // verified then |site_identity_details_| contains who verified the identity.
+  // This string will be displayed in the UI.
   string16 site_identity_details_;
 
   // Details about the connection to the website. In case of an encrypted
   // connection |site_connection_details_| contains encryption details, like
-  // encryption strength and ssl protocol version.
+  // encryption strength and ssl protocol version. This string will be
+  // displayed in the UI.
   string16 site_connection_details_;
 
-  // For websites that provided an EV certificate |orgainization_name_| contains
-  // the organization name of the certificate. In all other cases
-  // |organization_name| is an empty string.
+  // For websites that provided an EV certificate |orgainization_name_|
+  // contains the organization name of the certificate. In all other cases
+  // |organization_name| is an empty string. This string will be displayed in
+  // the UI.
   string16 organization_name_;
 
   // The |CertStore| provides all X509Certificates.
   content::CertStore* cert_store_;
 
   // The |HostContentSettingsMap| is the service that provides and manages
   // content settings (aka. site permissions).
   HostContentSettingsMap* content_settings_;
 
   DISALLOW_COPY_AND_ASSIGN(WebsiteSettings);
 };
 
 #endif  // CHROME_BROWSER_WEBSITE_SETTINGS_H_