Allow systematic prefix search in bookmarks.

Allow systematic prefix search in bookmarks.

This CL allows the bookmark model's search to always use prefix search, even
when terms are too short.

BUG=415774
Committed: https://crrev.com/f1a7a37376ffcf694b93845be75a313af6e61e24
Cr-Commit-Position: refs/heads/master@{#302598}

[Kibeom Kim (inactive), 2014-11-04 01:29:19]

kkimlabs@chromium.org changed reviewers:
+ ianwen@chromium.org, lpromero@chromium.org, noyau@chromium.org,
sky@chromium.org

[Kibeom Kim (inactive), 2014-11-04 01:29:20]

sky@: ptal

"Patch set 1" here is identical to the "Patch set 1" at
https://codereview.chromium.org/701553002/

[sky, 2014-11-04 03:32:05]

LGTM

https://codereview.chromium.org/703553002/diff/20001/components/query_parser/...
File components/query_parser/query_parser.h (right):

https://codereview.chromium.org/703553002/diff/20001/components/query_parser/...
components/query_parser/query_parser.h:132: const MatchingAlgorithm
matching_algorithm,
Are we really using const for these? Isn't the enum the same as any other POD
type in which we don't generally use const in function definition/declaration?

[Kibeom Kim (inactive), 2014-11-04 08:38:31]

https://codereview.chromium.org/703553002/diff/20001/components/query_parser/...
File components/query_parser/query_parser.h (right):

https://codereview.chromium.org/703553002/diff/20001/components/query_parser/...
components/query_parser/query_parser.h:132: const MatchingAlgorithm
matching_algorithm,
On 2014/11/04 03:32:05, sky wrote:
> Are we really using const for these? Isn't the enum the same as any other POD
> type in which we don't generally use const in function definition/declaration?

Done.

[Kibeom Kim (inactive), 2014-11-04 08:38:34]

The CQ bit was checked by kkimlabs@chromium.org

[commit-bot: I haz the power, 2014-11-04 08:39:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/703553002/60001

[Kibeom Kim (inactive), 2014-11-04 08:40:54]

The CQ bit was unchecked by kkimlabs@chromium.org

[Kibeom Kim (inactive), 2014-11-04 08:43:14]

The CQ bit was checked by kkimlabs@chromium.org

[commit-bot: I haz the power, 2014-11-04 08:44:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/703553002/80001

[commit-bot: I haz the power, 2014-11-04 10:30:53]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2014-11-04 10:32:15]

Patchset 5 (id:??) landed as
https://crrev.com/f1a7a37376ffcf694b93845be75a313af6e61e24
Cr-Commit-Position: refs/heads/master@{#302598}