[Sync] Change the number of URLs in the typed URLs perf test

[Sync] Change the number of URLs in the typed URLs perf test

This is a partial speculative fix to stabilize the run times of the delete
portion of the perf test.

BUG=107034, 107040
TEST=


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

[akalin, 2011-12-10 02:13:01]

+zea for review

[Andrew T Wilson (Slow), 2011-12-10 02:23:40]

On 2011/12/10 02:13:01, akalin wrote:
> +zea for review

LGTM

[Andrew T Wilson (Slow), 2011-12-10 02:23:46]

http://codereview.chromium.org/8892028/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/8892028/diff/1/chrome/browser/sync/test/integr...
chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc:22:
static const int kNumUrls = 163;
This is a little arbitrary, but OK. An alternative would've been some ugly
expression like:

kNumUrls = 150 - (150 % browser_sync::kDefaultMaxCommitBatchSize) +
browser_sync::kDefaultMaxCommitBatchSize/2;

...but I'm fine with this too.

[akalin, 2011-12-10 02:25:11]

On 2011/12/10 02:23:46, Andrew T Wilson wrote:
>
http://codereview.chromium.org/8892028/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/8892028/diff/1/chrome/browser/sync/test/integr...
>
chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc:22:
> static const int kNumUrls = 163;
> This is a little arbitrary, but OK. An alternative would've been some ugly
> expression like:
> 
> kNumUrls = 150 - (150 % browser_sync::kDefaultMaxCommitBatchSize) +
> browser_sync::kDefaultMaxCommitBatchSize/2;
> 
> ...but I'm fine with this too.

Yeah, I think having it be an arithmetic expression may create a static
initializer (not in this case since the constant is actually in the header file,
but if the definition of the constant gets moved into a .cc file like it should
be).

[commit-bot: I haz the power, 2011-12-10 02:29:54]

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

[commit-bot: I haz the power, 2011-12-10 04:21:37]

Change committed as 113941