Add ExpireHistoryBackend::DeleteURLs method

Add ExpireHistoryBackend::DeleteURLs method

This is primarily used by the sync typed URL performance tests.  This
method helps stabilize the perf numbers since there's only a single
notification that is emitted (instead of one per URL).

BUG=107040
TEST=


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

[akalin, 2011-12-14 23:00:18]

+sky for review

[sky, 2011-12-14 23:16:49]

http://codereview.chromium.org/8910024/diff/1/chrome/browser/history/expire_h...
File chrome/browser/history/expire_history_backend.cc (right):

http://codereview.chromium.org/8910024/diff/1/chrome/browser/history/expire_h...
chrome/browser/history/expire_history_backend.cc:200: if (!main_db_)
Implement this in terms of DeleteURLs.

[akalin, 2011-12-14 23:43:08]

PTAL

http://codereview.chromium.org/8910024/diff/1/chrome/browser/history/expire_h...
File chrome/browser/history/expire_history_backend.cc (right):

http://codereview.chromium.org/8910024/diff/1/chrome/browser/history/expire_h...
chrome/browser/history/expire_history_backend.cc:200: if (!main_db_)
On 2011/12/14 23:16:49, sky wrote:
> Implement this in terms of DeleteURLs.

Done.

[sky, 2011-12-14 23:54:02]

LGTM

[commit-bot: I haz the power, 2011-12-14 23:57:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/akalin@chromium.org/8910024/4003

[commit-bot: I haz the power, 2011-12-15 02:08:01]

Change committed as 114572

[sky, 2011-12-16 22:00:11]

http://codereview.chromium.org/8910024/diff/4003/chrome/browser/history/expir...
File chrome/browser/history/expire_history_backend.cc (right):

http://codereview.chromium.org/8910024/diff/4003/chrome/browser/history/expir...
chrome/browser/history/expire_history_backend.cc:212: return;  // Nothing to
delete.
This should be continue now.

http://codereview.chromium.org/8910024/diff/4003/chrome/browser/history/expir...
chrome/browser/history/expire_history_backend.cc:234:
DeleteFaviconsIfPossible(dependencies.affected_favicons);
This is wrong now. In particular if the previous url was bookmarked, this still
ends up deleting favicons.

http://codereview.chromium.org/8910024/diff/4003/chrome/browser/history/expir...
chrome/browser/history/expire_history_backend.cc:236: if (text_db_)
This should be outside the loop.