[dom_distiller] Add support for lookup by URL for InMemoryContentStore.

[dom_distiller] Add support for lookup by URL for InMemoryContentStore.

The InMemoryContentStore currently only supports lookup by entry ID, but this
CL adds support for also looking up articles by URLs. This will make the class
more usable for when users manually distill web pages.

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

[nyquist, 2014-06-03 19:25:50]

cjhopman: PTAL

[cjhopman, 2014-06-06 17:33:05]

https://codereview.chromium.org/314573005/diff/40001/components/dom_distiller...
File components/dom_distiller/core/distilled_content_store.cc (right):

https://codereview.chromium.org/314573005/diff/40001/components/dom_distiller...
components/dom_distiller/core/distilled_content_store.cc:77:
std::pair<std::string, std::string>(page.url(), entry.entry_id()));
how about just:
url_to_id_[page.url()] = entry.entry_id();

or at least:
url_to_id_.insert(std::make_pair(page.url(), entry.entry_id()));

https://codereview.chromium.org/314573005/diff/40001/components/dom_distiller...
File components/dom_distiller/core/distilled_content_store.h (right):

https://codereview.chromium.org/314573005/diff/40001/components/dom_distiller...
components/dom_distiller/core/distilled_content_store.h:81: typedef
base::hash_map<std::string, std::string> URLMap;
s/URLMap/UrlMap

[nyquist, 2014-06-06 19:00:21]

cjhopman: PTAL

https://codereview.chromium.org/314573005/diff/40001/components/dom_distiller...
File components/dom_distiller/core/distilled_content_store.cc (right):

https://codereview.chromium.org/314573005/diff/40001/components/dom_distiller...
components/dom_distiller/core/distilled_content_store.cc:77:
std::pair<std::string, std::string>(page.url(), entry.entry_id()));
On 2014/06/06 17:33:05, cjhopman wrote:
> how about just:
> url_to_id_[page.url()] = entry.entry_id();
> 
> or at least:
> url_to_id_.insert(std::make_pair(page.url(), entry.entry_id()));

Done.

https://codereview.chromium.org/314573005/diff/40001/components/dom_distiller...
File components/dom_distiller/core/distilled_content_store.h (right):

https://codereview.chromium.org/314573005/diff/40001/components/dom_distiller...
components/dom_distiller/core/distilled_content_store.h:81: typedef
base::hash_map<std::string, std::string> URLMap;
On 2014/06/06 17:33:05, cjhopman wrote:
> s/URLMap/UrlMap

Done.

[cjhopman, 2014-06-06 19:30:37]

lgtm

[nyquist, 2014-06-06 20:10:23]

The CQ bit was checked by nyquist@chromium.org

[commit-bot: I haz the power, 2014-06-06 20:12:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nyquist@chromium.org/314573005/60001

[commit-bot: I haz the power, 2014-06-07 05:04:53]

Change committed as 275618