[Sync] Add HistoryService::DeleteURLsForTest

[Sync] Add HistoryService::DeleteURLsForTest

This is so that the sync performance tests don't need to wait for each
URL delete to hit the disk before moving on to the next one.

This is a speculative fix to stabilize the delete_typed_urls sync perf
test.  It also cuts down the runtime of the typed urls test from >20s to <10s or so.

BUG=107040
TEST=

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

[akalin, 2011-12-14 00:45:14]

+sky for history stuff, +atwilson for sync stuff

[sky, 2011-12-14 15:31:26]

LGTM

[Andrew T Wilson (Slow), 2011-12-14 18:17:06]

LGTM

http://codereview.chromium.org/8890089/diff/1/chrome/browser/sync/test/integr...
File
chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc
(right):

http://codereview.chromium.org/8890089/diff/1/chrome/browser/sync/test/integr...
chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc:76:
const std::vector<history::URLRow>& urls = GetTypedUrlsFromClient(profile);
This scares me, but I trust you when you say this is guaranteed to work.

[akalin, 2011-12-14 18:21:15]

http://codereview.chromium.org/8890089/diff/1/chrome/browser/sync/test/integr...
File
chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc
(right):

http://codereview.chromium.org/8890089/diff/1/chrome/browser/sync/test/integr...
chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc:76:
const std::vector<history::URLRow>& urls = GetTypedUrlsFromClient(profile);
On 2011/12/14 18:17:07, Andrew T Wilson wrote:
> This scares me, but I trust you when you say this is guaranteed to work.

Yeap, see
http://herbsutter.com/2008/01/01/gotw-88-a-candidate-for-the-most-important-c...
.

I like it because it's the closest C++ can come to let bindings.