Render extension URLs with chips

chrome/browser/ui/location_bar/location_bar.h

Index: chrome/browser/ui/location_bar/location_bar.h
diff --git a/chrome/browser/ui/location_bar/location_bar.h b/chrome/browser/ui/location_bar/location_bar.h
index e3e316fac94e7605652021d8212970deb23f9a09..46c851882f8b1ca8500e5df229b56aacb01aff13 100644
--- a/chrome/browser/ui/location_bar/location_bar.h
+++ b/chrome/browser/ui/location_bar/location_bar.h
@@ -104,6 +104,12 @@ class LocationBar {
   // Checks if any extension has requested that the bookmark star be hidden.
   bool IsBookmarkStarHiddenByExtension() const;
 
+  // Returns the name of the extension associated with the page, if any.
+  // Returns empty string otherwise. Performs basic sanitization on the
+  // extension name.

[Peter Kasting, 2016/12/14 22:47:07]

Nit: "associated with the page" is kinda vague; if a content script is acting on
a page, is that "associated"?  Maybe say "If |url| is an extension URL, returns
the name of the associated extension, with whitespace collapsed.  |web_contents|
is used to get at the extension registry."

[meacer, 2016/12/16 18:46:17]

Done.

+  base::string16 GetExtensionName(const GURL& url,
+                                  content::WebContents* web_contents) const;

[Peter Kasting, 2016/12/14 22:47:07]

It seems like this method can be static, not just const.

[meacer, 2016/12/16 18:46:17]

Done.

+
  private:
   Profile* profile_;