Issue 7491062: Sync performance test: Remove disabled benchmark tests.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 7491062: Sync performance test: Remove disabled benchmark tests. (Closed)

Created:
9 years, 4 months ago by braffert

Modified:
9 years, 4 months ago

Reviewers:
anna

CC:
chromium-reviews, ncarter (slow), idana, Raghu Simha, Erik does not do reviews, mihaip+watch_chromium.org, Aaron Boodman, Paweł Hajdan Jr., tim (not reviewing)

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

Description

Sync performance test: Remove disabled benchmark tests. Currently each datatype included in the sync performance tests contains a disabled test which was intended to be run offline to observe how the performance scales according to the number of entries synced. We are not planning on enabling these tests, because they were never intended to be automated in the first place, so they should not be included at all. BUG=none TEST=sync_performance_tests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96441

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+0 lines, -251 lines)			Patch
M	chrome/test/live_sync/performance/autofill_sync_perf_test.cc	View	1 2	4 chunks	+0 lines, -50 lines	0 comments	Download
M	chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc	View	1 2	4 chunks	+0 lines, -53 lines	0 comments	Download
M	chrome/test/live_sync/performance/extensions_sync_perf_test.cc	View	1 2	4 chunks	+0 lines, -46 lines	0 comments	Download
M	chrome/test/live_sync/performance/passwords_sync_perf_test.cc	View	1 2	4 chunks	+0 lines, -51 lines	0 comments	Download
M	chrome/test/live_sync/performance/typed_urls_sync_perf_test.cc	View	1 2	4 chunks	+0 lines, -51 lines	0 comments	Download

Messages

Total messages: 2 (0 generated)

Expand Messages | Collapse Messages

End of an era.

LGTM.

Expand Messages | Collapse Messages

Issue 7491062: Sync performance test: Remove disabled benchmark tests. (Closed)

Description

Patch Set 1 #

Patch Set 2 : rebase #

Patch Set 3 : rebase again #

Messages