[omnibox] Narrow condition for resetting selection

components/omnibox/browser/omnibox_edit_model.cc

Index: components/omnibox/browser/omnibox_edit_model.cc
diff --git a/components/omnibox/browser/omnibox_edit_model.cc b/components/omnibox/browser/omnibox_edit_model.cc
index 3e195f8bb0ac10890e77d55f23da405a34e1a9ce..4d4db3648aa6e33d98efada9acc0d1e4da66b1ba 100644
--- a/components/omnibox/browser/omnibox_edit_model.cc
+++ b/components/omnibox/browser/omnibox_edit_model.cc
@@ -338,6 +338,11 @@ void OmniboxEditModel::AdjustTextForCopy(int sel_min,
   }
 }
 
+bool OmniboxEditModel::text_update_in_progress() const {
+  return permanent_text_ !=
+      controller_->GetToolbarModel()->GetFormattedURL(nullptr);

[Peter Kasting, 2017/03/21 22:47:18]

Why would the |permanent_text_| be out of sync with the toolbar model?

In OmniboxEditModel::UpdatePermanentText(), we unconditionally update
|permanent_text_|, and then we return whether the view should actually change
something visibly.  So I'd expect these two values to be in sync unless we
haven't been Update()d yet, in which case something else is funky.

I'd think this bug would be solvable by comparing PermanentURL() to the visible
text in the view, which can all be done on the view side.

[Kevin Bailey, 2017/03/23 13:14:37]

The comment in ::OnBlur() implies that some update gets delayed. I couldn't be
certain which delay that it was referring to, but I assumed it was this delay we
were trying to detect, and that permanent_url_ is downstream of what the user
typed.
::PermanentURL() includes 'https://.../'. I'm hoping that there is an
intermediate value that is more similar?

[Peter Kasting, 2017/03/23 21:52:12]

The paragraph I wrote describes the place this is referring to.  Basically, the
permanent_url_ should already be up-to-date, so this function you've added
should always return false.  (If that's not what you saw, then I'm curious about
how that could be.)
Did you mean http://?  (i.e., do you mean this includes a scheme that is
normally stripped when displaying the URL in the steady-state omnibox?)

You probably want to compare against the raw permanent_text_ instead, but I
don't think there's an accessor for it today.  You could add one.

[Kevin Bailey, 2017/04/10 18:49:30]

I haven't seen anything different. The code was my best guess at the
time.
I meant that it wasn't anything like the visible text. I didn't
realize that PermanentURL() mangled it so much.

+}
+
 void OmniboxEditModel::SetInputInProgress(bool in_progress) {
   if (in_progress && !user_input_since_focus_) {
     base::TimeTicks now = base::TimeTicks::Now();