[Mac] Always restyle omnibox after user input.

chrome/browser/autocomplete/autocomplete_edit_view_mac.mm

 // Copyright (c) 2009 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/autocomplete/autocomplete_edit_view_mac.h"
 
 #include "base/sys_string_conversions.h"
 #include "chrome/browser/autocomplete/autocomplete_edit.h"
 #include "chrome/browser/autocomplete/autocomplete_popup_model.h"
 #include "chrome/browser/autocomplete/autocomplete_popup_view_mac.h"
@@ skipping 473 matching lines @@
   // and other methods to provide positive knowledge that a delete
   // occured, rather than intuiting it from context.  Consider whether
   // that would be a stronger approach.
   const bool just_deleted_text =
       (length < text_before_change_.length() &&
        new_selection.location <= selection_before_change_.location);
 
   const bool something_changed = model_->OnAfterPossibleChange(new_text,
       selection_differs, text_differs, just_deleted_text, at_end_of_edit);
 
-  // Restyle if the user changed something.
-  // TODO(shess): Does this need to diver deeper to avoid flashing?
-  // For instance, if the user types "/" after the hostname, will it
-  // show as HostTextColor() for an instant before being replaced by
-  // BaseTextColor().  This could probably be done by subclassing the
-  // cell and intercepting draw requests so that we draw the right
-  // thing every time.
-  // NOTE(shess): That kind of thing would also help with when the
-  // flashing from when the user's typing replaces the selection which
-  // is then re-created with the new autocomplete results.
-  if (something_changed) {
-    // TODO(shess) I believe that this call is needless duplication of
-    // effort.  As best I can tell, something_changed can only be true
-    // if |model_| called UpdatePopup(), which then updated us.  But
-    // the state involved seems substantial.
-    EmphasizeURLComponents();
-  }
+  // Restyle in case the user changed something.
+  // TODO(shess): I believe there are multiple-redraw cases, here.
+  // Linux watches for something_changed && text_differs, but that
+  // fails for us in case you copy the URL and paste the identical URL
+  // back (we'll lose the styling).
+  EmphasizeURLComponents();
 
   return something_changed;
 }
 
 void AutocompleteEditViewMac::OnUpOrDownKeyPressed(bool up, bool by_page) {
   // We should only arrive here when the field is focussed.
   DCHECK(IsFirstResponder());
 
   const int count = by_page ? model_->result().size() : 1;
   model_->OnUpOrDownKeyPressed(up ? -count : count);
@@ skipping 118 matching lines @@
   // TODO(shess): Figure out where the selection belongs.  On GTK,
   // it's set to the start of the text.
 }
 
 // Signal that we've lost focus when the window resigns key.
 - (void)windowDidResignKey:(NSNotification*)notification {
   edit_view_->OnDidResignKey();
 }
 
 @end