Commit b947ed40 authored by Justin Donnelly's avatar Justin Donnelly Committed by Commit Bot

[omnibox] Fix incorrect display of user input after option-enter.

After an option-enter navigation we should be displaying the URL.

I'm a little nervous about changing this logic, but I've tried every
kind of search and navigation I can think of and they all seem to still
work correctly.

Bug: 410707
Change-Id: I0d30ce7e23d6b98e4c99857e5b8226364398b758
Reviewed-on: https://chromium-review.googlesource.com/935672Reviewed-by: default avatarTrent Apted <tapted@chromium.org>
Commit-Queue: Justin Donnelly <jdonnelly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541916}
parent a1fadc8f
......@@ -487,7 +487,9 @@ void OmniboxViewMac::EmphasizeURLComponents() {
// more. Calling -stringValue ensures that |field_| reflects the changes to
// |storage|.
[field_ stringValue];
} else {
} else if (!in_coalesced_update_block_) {
// Skip this if we're in a coalesced update block. Otherwise, the user text
// entered can get set in a new tab because we haven't yet set the URL text.
SetText(GetText());
}
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment