DescriptionOmnibox: Fix Inline Autocompletion of Navsuggest in Forced Query Mode
This moves the block of code that adds the "?" from one place to another.
Its original location was wrong because it incremented
|inline_autocomplete_offset| before passing it to FormatUrl().
This could change how FormatUrl() would correct inline_autocomplete_offset
when its value is near the end of the formatted string.
This is wrong.
In both locations, in effect we are adding the "?" prefix after FormatUrl()
runs. This means we should correct the inline_autocomplete_offset value
to compensate for the extra character "?" only after FormatUrl() runs.
This fixes the bug.
I also added a test that would've caught this issue.
BUG=313451
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233544
Patch Set 1 #Patch Set 2 : reupload #
Messages
Total messages: 4 (0 generated)
|