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 82fb098456682549969271da999e4042c19f8c32..3d6901f029b4e81b35af380e8e267045cab79500 100644 |
--- a/chrome/browser/ui/views/location_bar/location_bar_view.h |
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.h |
@@ -77,6 +77,7 @@ class LocationBarView : public LocationBar, |
public: |
// Should return the current web contents. |
virtual content::WebContents* GetWebContents() = 0; |
+ virtual const content::WebContents* GetWebContents() const = 0; |
Peter Kasting
2016/12/14 22:47:07
Not clear to me why this is useful, since your Get
meacer
2016/12/16 18:46:17
LocationBarView::GetPreferredSize is a const metho
Peter Kasting
2016/12/20 01:59:31
My point is, the const method you're adding return
meacer
2016/12/21 01:38:56
Done.
|
virtual ToolbarModel* GetToolbarModel() = 0; |
virtual const ToolbarModel* GetToolbarModel() const = 0; |
@@ -324,6 +325,10 @@ class LocationBarView : public LocationBar, |
// Returns true if the chip should be animated |
bool ShouldAnimateSecurityChip() const; |
+ // Returns true when the current page is a chrome-extension:// or a chrome:// |
+ // URL. |
Peter Kasting
2016/12/14 22:47:07
This seems inaccurate since we don't return true f
meacer
2016/12/16 18:46:18
Removed and folded into ShouldShowSecurityText.
|
+ bool ShouldShowExtensionChip() const; |
Peter Kasting
2016/12/14 22:47:07
Rather than adding this, and then having to add du
meacer
2016/12/16 18:46:17
Sure, done. I checked with the UI folks and they s
|
+ |
// 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 |