Shows chrome-extension urls and greys out the whole url.

chrome/browser/ui/views/omnibox/omnibox_view_win.cc

 // Copyright 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.
 
 #include "chrome/browser/ui/views/omnibox/omnibox_view_win.h"
 
 #include <algorithm>
 #include <locale>
 #include <string>
 
@@ skipping 2458 matching lines @@
   GetSelection(saved_sel);
 
   // See whether the contents are a URL with a non-empty host portion, which we
   // should emphasize.  To check for a URL, rather than using the type returned
   // by Parse(), ask the model, which will check the desired page transition for
   // this input.  This can tell us whether an UNKNOWN input string is going to
   // be treated as a search or a navigation, and is the same method the Paste
   // And Go system uses.
   url_parse::Component scheme, host;
   AutocompleteInput::ParseForEmphasizeComponents(GetText(), &scheme, &host);
   const bool emphasize = model()->CurrentTextIsURL() && (host.len > 0);

[Peter Kasting, 2013/04/04 21:07:05]

Nit: Remove this, it's unused

[Patrick Riordan, 2013/04/10 02:00:24]

Done.

 
   // Set the baseline emphasis.
   CHARFORMAT cf = {0};
   cf.dwMask = CFM_COLOR;
   // If we're going to emphasize parts of the text, then the baseline state
   // should be "de-emphasized".  If not, then everything should be rendered in
-  // the standard text color.
-  cf.crTextColor = skia::SkColorToCOLORREF(parent_view_->GetColor(
-      security_level_,
-      emphasize ? LocationBarView::DEEMPHASIZED_TEXT : LocationBarView::TEXT));
-  // NOTE: Don't use SetDefaultCharFormat() instead of the below; that sets the
-  // format that will get applied to text added in the future, not to text
-  // already in the edit.
-  SelectAll(false);
-  SetSelectionCharFormat(cf);
+  // the standard text color unless we should grey out the entire URL.
+  if (model()->CurrentTextIsURL() &&

[Peter Kasting, 2013/04/04 21:07:05]

Nit: Slightly simpler:

  bool grey_base = model()->CurrentTextIsURL() &&
      (host.is_nonempty() || toolbar_model()->ShouldGreyOutURL());
  cf.crTextColor = skia::SkColorToCOLORREF(parent_view_->GetColor(
      security_level_,
      grey_base ? LocationBarView::DEEMPHASIZED_TEXT : LocationBarView::TEXT));
  // NOTE: Don't use SetDefaultCharFormat() instead of the below; that sets the
  // format that will get applied to text added in the future, not to text
  // already in the edit.
  SelectAll(false);
  SetSelectionCharFormat(cf);
  if (grey_base && !toolbar_model()->ShouldGreyOutURL()) {
    // We've found a host name and we should provide emphasis to host names, so
    // emphasize it.
    cf.crTextColor = skia::SkColorToCOLORREF(parent_view_->GetColor(
        security_level_, LocationBarView::TEXT));
    SetSelectionCharFormat(cf);
  }

[Patrick Riordan, 2013/04/10 02:00:24]

Done.

+      (host.is_nonempty() || toolbar_model()->ShouldGreyOutURL())) {
+    cf.crTextColor = skia::SkColorToCOLORREF(parent_view_->GetColor(
+        security_level_, LocationBarView::DEEMPHASIZED_TEXT));
+    // NOTE: Don't use SetDefaultCharFormat() instead of the below; that sets
+    // the format that will get applied to text added in the future, not to text
+    // already in the edit.
+    SelectAll(false);
+    SetSelectionCharFormat(cf);
 
-  if (emphasize) {
-    // We've found a host name, give it more emphasis.
+    if (host.is_nonempty() && !toolbar_model()->ShouldGreyOutURL()) {
+      // We've found a host name and we should provide emphasis to host names,
+      // so emphasize it.
+      cf.crTextColor = skia::SkColorToCOLORREF(parent_view_->GetColor(
+          security_level_, LocationBarView::TEXT));
+      SetSelection(host.begin, host.end());
+      SetSelectionCharFormat(cf);
+    }
+  } else {
     cf.crTextColor = skia::SkColorToCOLORREF(parent_view_->GetColor(
         security_level_, LocationBarView::TEXT));
-    SetSelection(host.begin, host.end());
+    // NOTE: Don't use SetDefaultCharFormat(). See above.
+    SelectAll(false);
     SetSelectionCharFormat(cf);
   }
 
   // Emphasize the scheme for security UI display purposes (if necessary).
   insecure_scheme_component_.reset();
   if (!model()->user_input_in_progress() && model()->CurrentTextIsURL() &&
       scheme.is_nonempty() && (security_level_ != ToolbarModel::NONE)) {
     if (security_level_ == ToolbarModel::SECURITY_ERROR) {
       insecure_scheme_component_.begin = scheme.begin;
       insecure_scheme_component_.len = scheme.len;
@@ skipping 346 matching lines @@
   return (rect.left - client_rect.left) + (client_rect.right - rect.right);
 }
 
 int OmniboxViewWin::WidthNeededToDisplay(const string16& text) const {
   // Use font_.GetStringWidth() instead of
   // PosFromChar(location_entry_->GetTextLength()) because PosFromChar() is
   // apparently buggy. In both LTR UI and RTL UI with left-to-right layout,
   // PosFromChar(i) might return 0 when i is greater than 1.
   return font_.GetStringWidth(text) + GetHorizontalMargin();
 }