Omnibox: Trim Leading Spaces After Initial Forced-Query '?'

Omnibox: Trim Leading Spaces After Initial Forced-Query '?'

BUG=170727
TEST=doesn't crash

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=177957

[Peter Kasting, 2013-01-21 18:53:15]

Make sure the following still works correctly:

* Search for "foo bar"
* Double-check that "foo b" will now inline-autocomplete
* Type "? foo b"
* Ensure the autocompletion is "ar", not "r" or something

https://codereview.chromium.org/11896016/diff/1/chrome/browser/autocomplete/a...
File chrome/browser/autocomplete/autocomplete_input.cc (right):

https://codereview.chromium.org/11896016/diff/1/chrome/browser/autocomplete/a...
chrome/browser/autocomplete/autocomplete_input.cc:76: // Remove spaces between
opening questions mark and first actual character.
Nit: questions -> question

[Peter Kasting, 2013-01-21 18:53:21]

LGTM

[Peter Kasting, 2013-01-21 18:53:40]

Sigh, three messages... forgot to ask if this could get a test too

[Mark P, 2013-01-21 20:50:54]

Added unit tests.

Also did the manual test you requested.  Works fine.

Submitting.

--mark

https://codereview.chromium.org/11896016/diff/1/chrome/browser/autocomplete/a...
File chrome/browser/autocomplete/autocomplete_input.cc (right):

https://codereview.chromium.org/11896016/diff/1/chrome/browser/autocomplete/a...
chrome/browser/autocomplete/autocomplete_input.cc:76: // Remove spaces between
opening questions mark and first actual character.
On 2013/01/21 18:53:15, Peter Kasting wrote:
> Nit: questions -> question

Done.

[commit-bot: I haz the power, 2013-01-21 20:51:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mpearson@chromium.org/11896016/5001

[commit-bot: I haz the power, 2013-01-21 23:23:28]

Change committed as 177957