Part 2 of the SSL InfoBubble. 2nd try, this time with test fixed.

- Added Certificate Information link.

- Changed iconography to be:
 Identity  : Green check mark vs. Red skull and bones
 Connection: Green padlock vs. Orange exclamation point.
 (This used to be Green checkmark and Orange Exclamation point for both sections.

- Removed the icon for the First visit section. Need to see if we want an icon for when you haven't visited the site before. Mocks show no icons when you have visited before.

BUG=52916
TEST=Open the SSL InfoBubble and notice the new iconography. Click the link and the Certificate dialog should show.

Review URL: http://codereview.chromium.org/3187019

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

1 files changed, 18 insertions, 6 deletions

diff --git a/chrome/browser/page_info_model.h b/chrome/browser/page_info_model.h
index 6bf0389..60e3883 100644
--- a/chrome/browser/page_info_model.h
+++ b/chrome/browser/page_info_model.h
@@ -18,7 +18,7 @@ class PrefService;
 class Profile;
 
 // The model that provides the information that should be displayed in the page
-// info dialog.
+// info dialog/bubble.
 class PageInfoModel {
  public:
   class PageInfoModelObserver {
@@ -29,15 +29,23 @@ class PageInfoModel {
     virtual ~PageInfoModelObserver() {}
   };
 
+  enum SectionInfoType {
+    SECTION_INFO_IDENTITY = 0,
+    SECTION_INFO_CONNECTION,
+    SECTION_INFO_FIRST_VISIT,
+  };
+
   struct SectionInfo {
     SectionInfo(bool state,
                 const string16& title,
-                const string16& head_line,
-                const string16& description)
+                const string16& headline,
+                const string16& description,
+                SectionInfoType type)
         : state(state),
           title(title),
-          head_line(head_line),
-          description(description) {
+          headline(headline),
+          description(description),
+          type(type) {
     }
 
     bool state;  // True if state is OK, false otherwise (ex of bad states:
@@ -47,10 +55,14 @@ class PageInfoModel {
     string16 title;
 
     // A single line describing the section, optional.
-    string16 head_line;
+    string16 headline;
 
     // The full description of what this section is.
     string16 description;
+
+    // The type of SectionInfo we are dealing with, for example: Identity,
+    // Connection, First Visit.
+    SectionInfoType type;
   };
 
   PageInfoModel(Profile* profile,