Add comment to explain why some of the strings are typed as string16, and a...

Add comment to explain why some of the strings are typed as string16, and a TODO as a reminder to change this down the road.

BUG=113688
TEST=none

Review URL: https://chromiumcodereview.appspot.com/10181004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133649 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/website_settings.h

@@ -147,18 +147,25 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver {
   // 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
+  // unnecessary UTF-8 string conversions.
+
   // 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.