Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9214)

Unified Diff: chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc

Issue 7541052: Sync performance tests: append underscore to private variable names. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/test/live_sync/performance/typed_urls_sync_perf_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc
diff --git a/chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc b/chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc
index e13b9b07416007344cb66d611ae368211247c9a5..9e932fff942f02bbff4b6a839f0b6f29429fe217 100644
--- a/chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc
+++ b/chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc
@@ -19,8 +19,8 @@ class BookmarksSyncPerfTest : public LiveSyncTest {
public:
BookmarksSyncPerfTest()
: LiveSyncTest(TWO_CLIENT),
- url_number(0),
- url_title_number(0) {}
+ url_number_(0),
+ url_title_number_(0) {}
// Adds |num_urls| new unique bookmarks to the bookmark bar for |profile|.
void AddURLs(int profile, int num_urls);
@@ -45,8 +45,8 @@ class BookmarksSyncPerfTest : public LiveSyncTest {
// Returns a new unique bookmark title.
std::wstring NextIndexedURLTitle();
- int url_number;
- int url_title_number;
+ int url_number_;
+ int url_title_number_;
DISALLOW_COPY_AND_ASSIGN(BookmarksSyncPerfTest);
};
@@ -88,11 +88,11 @@ void BookmarksSyncPerfTest::Cleanup() {
}
std::string BookmarksSyncPerfTest::NextIndexedURL() {
- return BookmarksHelper::IndexedURL(url_number++);
+ return BookmarksHelper::IndexedURL(url_number_++);
}
std::wstring BookmarksSyncPerfTest::NextIndexedURLTitle() {
- return BookmarksHelper::IndexedURLTitle(url_title_number++);
+ return BookmarksHelper::IndexedURLTitle(url_title_number_++);
}
IN_PROC_BROWSER_TEST_F(BookmarksSyncPerfTest, P0) {
« no previous file with comments | « no previous file | chrome/test/live_sync/performance/typed_urls_sync_perf_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698