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

Unified Diff: chrome/browser/history/url_database_unittest.cc

Issue 274040: More style nits. (Closed)
Patch Set: Created 11 years, 2 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
Index: chrome/browser/history/url_database_unittest.cc
diff --git a/chrome/browser/history/url_database_unittest.cc b/chrome/browser/history/url_database_unittest.cc
index 0f0be545d04914cae1fd82aca16b9ac7de7e1f1b..d6dc43fae05ca5b715d4fa338f1a35ddd46d9237 100644
--- a/chrome/browser/history/url_database_unittest.cc
+++ b/chrome/browser/history/url_database_unittest.cc
@@ -112,13 +112,13 @@ TEST_F(URLDatabaseTest, AddURL) {
EXPECT_EQ(0, GetRowForURL(GURL("http://news.google.com/"), &info));
// Delete all urls in the domain
- // FIXME(ACW) test the new url based delete domain
- //EXPECT_TRUE(db.DeleteDomain(kDomainID));
+ // TODO(acw): test the new url based delete domain
+ // EXPECT_TRUE(db.DeleteDomain(kDomainID));
// Make sure the urls have been properly removed
- // FIXME(ACW) commented out because remove no longer works.
- //EXPECT_TRUE(db.GetURLInfo(url1, NULL) == NULL);
- //EXPECT_TRUE(db.GetURLInfo(url2, NULL) == NULL);
+ // TODO(acw): commented out because remove no longer works.
+ // EXPECT_TRUE(db.GetURLInfo(url1, NULL) == NULL);
+ // EXPECT_TRUE(db.GetURLInfo(url2, NULL) == NULL);
}
// Tests adding, querying and deleting keyword visits.

Powered by Google App Engine
This is Rietveld 408576698