Omnibox: HistoryQuick Provider: Index Farther Into URL and Title

Omnibox: HistoryQuick Provider: Index Farther Into URL and Title

Record word break information for the first 200 characters in a URL or
title, not the first 50 characters.  This is important because we
generally drop URL suggestions that we think do not match at a word
break.

Tested locally on a longer URL from the bug.

TBR=sky
(because this change is trivial)

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

[Mark P, 2014-05-29 18:31:19]

PTAL.

thanks,
mark

[Peter Kasting, 2014-05-29 18:33:48]

LGTM

[Peter Kasting, 2014-05-29 18:34:13]

Do we need to flush any client databases or anything?

[Mark P, 2014-05-29 18:43:25]

On 2014/05/29 18:34:13, Peter Kasting wrote:
> Do we need to flush any client databases or anything?

Doh.  I thought about this three hours when discussing how hard it would be
experiment on this change, then forgot to put it in the changelist.  Now done. 
Please take another look.

Next up: add a test.

--mark

[Peter Kasting, 2014-05-29 18:44:13]

On 2014/05/29 18:43:25, Mark P wrote:
> Please take another look.

You also forgot to upload your new changelist.  :)

[Mark P, 2014-05-29 18:46:08]

On 2014/05/29 18:43:25, Mark P wrote:
> On 2014/05/29 18:34:13, Peter Kasting wrote:
> > Do we need to flush any client databases or anything?
> 
> Doh.  I thought about this three hours when discussing how hard it would be
> experiment on this change, then forgot to put it in the changelist.  Now done.

> Please take another look.

Patch now uploaded.

> Next up: add a test.

Ignore this comment.  I was thinking about a different changelist.  I think this
changelist is complete.

--mark

[Peter Kasting, 2014-05-29 18:46:50]

LGTM again

[Mark P, 2014-05-29 18:47:10]

The CQ bit was checked by mpearson@chromium.org

[commit-bot: I haz the power, 2014-05-29 18:48:14]

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

[commit-bot: I haz the power, 2014-05-29 22:12:13]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[commit-bot: I haz the power, 2014-05-29 22:15:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-29 22:15:29]

Try jobs failed on following builders:
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)

[Mark P, 2014-05-29 23:36:53]

The CQ bit was checked by mpearson@chromium.org

[commit-bot: I haz the power, 2014-05-29 23:39:58]

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

[commit-bot: I haz the power, 2014-05-30 04:32:21]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/18...)

[commit-bot: I haz the power, 2014-05-30 04:58:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-30 04:58:56]

Try jobs failed on following builders:
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/18...)

[Mark P, 2014-05-30 13:50:33]

The CQ bit was checked by mpearson@chromium.org

[commit-bot: I haz the power, 2014-05-30 13:51:32]

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

[commit-bot: I haz the power, 2014-05-30 15:07:56]

Change committed as 273837