New location security strings and animation.

chrome/browser/ui/views/location_bar/location_bar_view.h

Index: chrome/browser/ui/views/location_bar/location_bar_view.h
diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.h b/chrome/browser/ui/views/location_bar/location_bar_view.h
index dea93ce21b77d5c7560620f7530fbe541146854a..9e1b63f7f7b81c1b177ae6c418315e87ebf1dfdb 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.h
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.h
@@ -109,7 +109,7 @@ class LocationBarView : public LocationBar,
     TEXT,
     SELECTED_TEXT,
     DEEMPHASIZED_TEXT,
-    EV_BUBBLE_TEXT_AND_BORDER,
+    SECURITY_CHIP_TEXT,
   };
 
   // The location bar view's class name.
@@ -318,9 +318,16 @@ class LocationBarView : public LocationBar,
   // Returns true if the suggest text is valid.
   bool HasValidSuggestText() const;
 
+  // Returns text describing the URL's security level, to be placed in the
+  // security chip.
+  base::string16 GetSecurityText() const;

[Peter Kasting, 2016/08/26 20:59:12]

Nit: Blank line below

[Kevin Bailey, 2016/08/29 13:38:10]

Done.

   bool ShouldShowKeywordBubble() const;
   bool ShouldShowEVBubble() const;
 
+  // Returns true if the security chip should be displayed (if it has a
+  // verified security level.)

[Peter Kasting, 2016/08/26 20:59:12]

Nit: As a reader I'm not really sure what "verified" means.  Maybe something
along the lines of "Returns true when the current page is explicitly secure or
insecure.  In these cases we should show this state in the security chip."

[Kevin Bailey, 2016/08/29 13:38:10]

Done.

+  bool ShouldShowSecurityChip() const;
+
   // Used to "reverse" the URL showing/hiding animations, since we use separate
   // animations whose curves are not true inverses of each other.  Based on the
   // current position of the omnibox, calculates what value the desired