Show scheme in black and content in gray for data: protocol urls

components/omnibox/browser/omnibox_view.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file defines helper functions shared by the various implementations
 // of OmniboxView.
 
 #include "components/omnibox/browser/omnibox_view.h"
 
 #include <utility>
@@ skipping 167 matching lines @@
   if (client) {
     model_.reset(new OmniboxEditModel(this, controller, std::move(client)));
   }
 }
 
 void OmniboxView::TextChanged() {
   EmphasizeURLComponents();
   if (model_.get())
     model_->OnChanged();
 }
+
+OmniboxView::DEEMPHASIZE_COMPONENTS OmniboxView::GetDeemphasis(
+    const base::string16& url_scheme,
+    const bool has_host) const {
+  OmniboxView::DEEMPHASIZE_COMPONENTS deemphasize = NOTHING;

[Peter Kasting, 2017/02/24 01:53:40]

Nit: Can remove this variable, change assignments below to direct returns,
remove "else"s, and "return NOTHING;" at the end of the function.

This also allows putting blank lines between each return and the next block, and
in general avoiding comments in between "if" and "else", which would be a
nice-to-have.

+
+  // This constant is copied from extensions/common/constants.h to avoid
+  // complicated dependencies.

[Peter Kasting, 2017/02/24 01:53:40]

Urgh.

Can we not have extensions/common/ in components/omnibox/DEPS?

If not, I would consider having the caller pass this in to avoid this.

+  const char kExtensionScheme[] = "chrome-extension";
+
+  if (model_->CurrentTextIsURL()) {
+    // Extension IDs are not human-readable, so deemphasize everything to draw
+    // attention to the human-readable name in the location icon text.
+    if (url_scheme == base::UTF8ToUTF16(kExtensionScheme))
+      deemphasize = EVERYTHING;
+    // Data URLs are rarely human-readable and can be used for spoofing, so draw
+    // attention to the scheme to emphasize "this is just a bunch of data".
+    else if (url_scheme == base::UTF8ToUTF16(url::kDataScheme))
+      deemphasize = ALL_BUT_SCHEME;
+    // For normal URLs, the host is the best proxy for "identity".
+    else if (has_host)
+      deemphasize = ALL_BUT_HOST;
+  }
+
+  return deemphasize;
+}