diff options
| author | mpearson@chromium.org <mpearson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-12 16:54:38 +0000 |
|---|---|---|
| committer | mpearson@chromium.org <mpearson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-12 16:54:38 +0000 |
| commit | 9ba666117ed56d6ff4b1e058f9027798faa31b22 (patch) | |
| tree | 3dfc455a13d18575206cc7259f0b5691e0bae56f /chrome/browser/invalidation/invalidation_service_factory_browsertest.cc | |
| parent | dd49d9c1c18e3b8639d4adea15e964cfa7bef62e (diff) | |
| download | chromium_src-9ba666117ed56d6ff4b1e058f9027798faa31b22.zip chromium_src-9ba666117ed56d6ff4b1e058f9027798faa31b22.tar.gz chromium_src-9ba666117ed56d6ff4b1e058f9027798faa31b22.tar.bz2 | |
Omnibox: append old matches, don't attempt to insert them
Make AddMatch() append old matches rather than attempt to insert them
in order. We call SortAndCull() after AddMatch() in the only code
path that uses AddMatch(), so the order doesn't matter.
Then eliminate AddMatch() because the revised version is short,
instead inlining the few things it still does in its only caller.
In some quick interactive testing, I didn't notice any problems.
The autocomplete_result tests still pass also. (That said, I don't
think we test CopyOldMatches that extensively in it.)
BUG=380357
Review URL: https://codereview.chromium.org/329783003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276714 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/invalidation/invalidation_service_factory_browsertest.cc')
0 files changed, 0 insertions, 0 deletions