Move InMemoryURLIndex into chrome/browser/autocomplete

Move InMemoryURLIndex into chrome/browser/autocomplete

InMemoryURLIndex is not used in the history code (except for the creation)
but only by the autocomplete code, so move the code there and all its
related types.

BUG=453790
Committed: https://crrev.com/039209fcd5598340b3c3b02d98f5453013953cad
Cr-Commit-Position: refs/heads/master@{#319634}

[sdefresne, 2015-02-27 18:35:24]

sdefresne@chromium.org changed reviewers:
+ mpearson@chromium.org

[sdefresne, 2015-02-27 18:35:25]

I think this class (and supporting code) does not belongs into the history code
but more into the autocomplete code.

As can be seen, only one file in chrome/browser/history ends up including any of
the files moved into chrome/browser/autocomplete.

I'll send some followup CLs to:
- revert the ScoredHistoryMatch::Builder changes
- remove the history namespace from those types
- remove the dependency of HistoryService on InMemoryURLIndex

Please take a look at this first step.

[Mark P, 2015-02-27 20:39:12]

lgtm

https://codereview.chromium.org/963823003/diff/1/chrome/browser/history/DEPS
File chrome/browser/history/DEPS (left):

https://codereview.chromium.org/963823003/diff/1/chrome/browser/history/DEPS#...
chrome/browser/history/DEPS:30:
"!components/bookmarks/browser/bookmark_utils.h",
This isn't a consequence of this change, right?  It's just something that
could've happened earlier with other changes I think.

[sdefresne, 2015-03-02 08:53:03]

Thank you for the review.

https://codereview.chromium.org/963823003/diff/1/chrome/browser/history/DEPS
File chrome/browser/history/DEPS (left):

https://codereview.chromium.org/963823003/diff/1/chrome/browser/history/DEPS#...
chrome/browser/history/DEPS:30:
"!components/bookmarks/browser/bookmark_utils.h",
On 2015/02/27 20:39:12, Mark P wrote:
> This isn't a consequence of this change, right?  It's just something that
> could've happened earlier with other changes I think.

Yes, it was a consequence of introducing ScoredHistoryMatch::Builder, so it is
not a consequence of the current CL.

[sdefresne, 2015-03-02 15:00:03]

sdefresne@chromium.org changed reviewers:
+ jcivelli@chromium.org, sky@chromium.org

[sdefresne, 2015-03-02 15:00:03]

jcivelli: (will TBR) OWNERS for
  chrome/test/base/history_index_restore_observer.h

sky: OWNERS for
  chrome/browser/BUILD.gn
  chrome/browser/autocomplete/BUILD.gn
  chrome/browser/chromeos/BUILD.gn
  chrome/browser/extensions/BUILD.gn
  chrome/browser/ui/BUILD.gn

[Jay Civelli, 2015-03-02 16:49:48]

LGTM for chrome/test

[sky, 2015-03-02 18:09:39]

sky@chromium.org changed reviewers:
+ brettw@chromium.org
- sky@chromium.org

[sky, 2015-03-02 18:09:40]

sky->brettw

[sdefresne, 2015-03-04 15:56:28]

brettw: ping for BUILD.gn OWNERS

[sdefresne, 2015-03-05 10:52:12]

brettw: please review the following files only
  chrome/browser/BUILD.gn
  chrome/browser/autocomplete/BUILD.gn
  chrome/browser/chromeos/BUILD.gn
  chrome/browser/extensions/BUILD.gn
  chrome/browser/ui/BUILD.gn

If you do not have time to review this, can I ask another OWNERS of //chrome to
review them?

BTW, there are no wildcard OWNERS for the BUILD.gn files as they are for the
.gyp/.gypi files in //chrome. Is this expected or just nobody thought of adding
them?

[sdefresne, 2015-03-06 09:05:35]

sdefresne@chromium.org changed reviewers:
+ jhawkins@google.com
- brettw@chromium.org

[sdefresne, 2015-03-06 09:05:36]

-brettw who appears to be unavailable
+jhawkins can you review
  chrome/browser/BUILD.gn
  chrome/browser/chromeos/BUILD.gn
  chrome/browser/extensions/BUILD.gn
  chrome/browser/ui/BUILD.gn

as there are no wildcard OWNERS for .gn files. Thank you.

[sky, 2015-03-06 17:49:44]

sky@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2015-03-06 17:49:45]

LGTM

[sdefresne, 2015-03-09 13:41:14]

Patchset #13 (id:240001) has been deleted

[sdefresne, 2015-03-09 13:44:23]

Patchset #10 (id:180001) has been deleted

[sdefresne, 2015-03-09 13:44:40]

Patchset #10 (id:200001) has been deleted

[sdefresne, 2015-03-09 13:44:51]

Patchset #10 (id:220001) has been deleted

[sdefresne, 2015-03-09 13:56:18]

Patchset #9 (id:160001) has been deleted

[sdefresne, 2015-03-09 15:17:09]

The CQ bit was checked by sdefresne@chromium.org

[sdefresne, 2015-03-09 15:17:09]

The patchset sent to the CQ was uploaded after l-g-t-m from
mpearson@chromium.org, jcivelli@chromium.org, sky@chromium.org
Link to the patchset: https://codereview.chromium.org/963823003/#ps260001
(title: "Fixing win_chromium_x64_rel_ng build")

[commit-bot: I haz the power, 2015-03-09 15:17:21]

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

[commit-bot: I haz the power, 2015-03-09 15:33:29]

Committed patchset #9 (id:260001)

[commit-bot: I haz the power, 2015-03-09 15:34:21]

Patchset 9 (id:??) landed as
https://crrev.com/039209fcd5598340b3c3b02d98f5453013953cad
Cr-Commit-Position: refs/heads/master@{#319634}