Makes the in memory db update rows that have search terms associated

Makes the in memory db update rows that have search terms associated
with them.

BUG=none
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=70777

[brettw, 2011-01-06 20:43:34]

http://codereview.chromium.org/6135001/diff/2001/chrome/browser/history/in_me...
File chrome/browser/history/in_memory_history_backend.cc (right):

http://codereview.chromium.org/6135001/diff/2001/chrome/browser/history/in_me...
chrome/browser/history/in_memory_history_backend.cc:94:
HasKeyword(visited_details->row.url())) {
Doesn't the row here have the keyword details as well? It sucks to have to query
the DB every time for this.

[sky, 2011-01-06 20:47:26]

On Thu, Jan 6, 2011 at 12:43 PM,  <brettw@chromium.org> wrote:
>
>
http://codereview.chromium.org/6135001/diff/2001/chrome/browser/history/in_me...
> File chrome/browser/history/in_memory_history_backend.cc (right):
>
>
http://codereview.chromium.org/6135001/diff/2001/chrome/browser/history/in_me...
> chrome/browser/history/in_memory_history_backend.cc:94:
> HasKeyword(visited_details->row.url())) {
> Doesn't the row here have the keyword details as well?

No. The keyword is maintained in a separate table that is indexed by url_id.

> It sucks to have
> to query the DB every time for this.

Indeed. On the positive side this is the in memory db so it shouldn't
be that bad.

  -Scott

[brettw, 2011-01-07 18:18:26]

LGTM