Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(40)

Side by Side Diff: chrome/browser/ui/location_bar/location_bar.h

Issue 2555063003: Render extension URLs with chips (Closed)
Patch Set: Enlarge the window so that it's over the minimum window size Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_LOCATION_BAR_LOCATION_BAR_H_ 5 #ifndef CHROME_BROWSER_UI_LOCATION_BAR_LOCATION_BAR_H_
6 #define CHROME_BROWSER_UI_LOCATION_BAR_LOCATION_BAR_H_ 6 #define CHROME_BROWSER_UI_LOCATION_BAR_LOCATION_BAR_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 virtual LocationBarTesting* GetLocationBarForTesting() = 0; 97 virtual LocationBarTesting* GetLocationBarForTesting() = 0;
98 98
99 Profile* profile() { return profile_; } 99 Profile* profile() { return profile_; }
100 100
101 protected: 101 protected:
102 virtual ~LocationBar(); 102 virtual ~LocationBar();
103 103
104 // Checks if any extension has requested that the bookmark star be hidden. 104 // Checks if any extension has requested that the bookmark star be hidden.
105 bool IsBookmarkStarHiddenByExtension() const; 105 bool IsBookmarkStarHiddenByExtension() const;
106 106
107 // Returns the name of the extension associated with the page, if any.
108 // Returns empty string otherwise. Performs basic sanitization on the
109 // extension name.
Peter Kasting 2016/12/14 22:47:07 Nit: "associated with the page" is kinda vague; if
meacer 2016/12/16 18:46:17 Done.
110 base::string16 GetExtensionName(const GURL& url,
111 content::WebContents* web_contents) const;
Peter Kasting 2016/12/14 22:47:07 It seems like this method can be static, not just
meacer 2016/12/16 18:46:17 Done.
112
107 private: 113 private:
108 Profile* profile_; 114 Profile* profile_;
109 115
110 DISALLOW_COPY_AND_ASSIGN(LocationBar); 116 DISALLOW_COPY_AND_ASSIGN(LocationBar);
111 }; 117 };
112 118
113 class LocationBarTesting { 119 class LocationBarTesting {
114 public: 120 public:
115 // Returns the total number of page actions in the Omnibox. 121 // Returns the total number of page actions in the Omnibox.
116 virtual int PageActionCount() = 0; 122 virtual int PageActionCount() = 0;
(...skipping 11 matching lines...) Expand all
128 virtual void TestPageActionPressed(size_t index) = 0; 134 virtual void TestPageActionPressed(size_t index) = 0;
129 135
130 // Returns whether or not the bookmark star decoration is visible. 136 // Returns whether or not the bookmark star decoration is visible.
131 virtual bool GetBookmarkStarVisibility() = 0; 137 virtual bool GetBookmarkStarVisibility() = 0;
132 138
133 protected: 139 protected:
134 virtual ~LocationBarTesting() {} 140 virtual ~LocationBarTesting() {}
135 }; 141 };
136 142
137 #endif // CHROME_BROWSER_UI_LOCATION_BAR_LOCATION_BAR_H_ 143 #endif // CHROME_BROWSER_UI_LOCATION_BAR_LOCATION_BAR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698