| 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 32ded0c6dc9b536081638fd2dc7597f9e68012f6..f56784422cea57414fbc27d6a33330312691dc82 100644
|
| --- a/chrome/browser/history/url_database_unittest.cc
|
| +++ b/chrome/browser/history/url_database_unittest.cc
|
| @@ -1,4 +1,4 @@
|
| -// Copyright (c) 2009 The Chromium Authors. All rights reserved.
|
| +// Copyright (c) 2010 The Chromium Authors. All rights reserved.
|
| // Use of this source code is governed by a BSD-style license that can be
|
| // found in the LICENSE file.
|
|
|
| @@ -67,9 +67,9 @@ class URLDatabaseTest : public testing::Test,
|
| sql::Connection db_;
|
| };
|
|
|
| -// Test add and query for the URL table in the HistoryDatabase
|
| +// Test add and query for the URL table in the HistoryDatabase.
|
| TEST_F(URLDatabaseTest, AddURL) {
|
| - // first, add two URLs
|
| + // First, add two URLs.
|
| const GURL url1("http://www.google.com/");
|
| URLRow url_info1(url1);
|
| url_info1.set_title(UTF8ToUTF16("Google"));
|
| @@ -88,7 +88,7 @@ TEST_F(URLDatabaseTest, AddURL) {
|
| url_info2.set_hidden(true);
|
| EXPECT_TRUE(AddURL(url_info2));
|
|
|
| - // query both of them
|
| + // Query both of them.
|
| URLRow info;
|
| EXPECT_TRUE(GetRowForURL(url1, &info));
|
| EXPECT_TRUE(IsURLRowEqual(url_info1, info));
|
| @@ -96,7 +96,7 @@ TEST_F(URLDatabaseTest, AddURL) {
|
| EXPECT_TRUE(id2);
|
| EXPECT_TRUE(IsURLRowEqual(url_info2, info));
|
|
|
| - // update the second
|
| + // Update the second.
|
| url_info2.set_title(UTF8ToUTF16("Google Mail Too"));
|
| url_info2.set_visit_count(4);
|
| url_info2.set_typed_count(1);
|
| @@ -104,19 +104,19 @@ TEST_F(URLDatabaseTest, AddURL) {
|
| url_info2.set_hidden(false);
|
| EXPECT_TRUE(UpdateURLRow(id2, url_info2));
|
|
|
| - // make sure it got updated
|
| + // Make sure it got updated.
|
| URLRow info2;
|
| EXPECT_TRUE(GetRowForURL(url2, &info2));
|
| EXPECT_TRUE(IsURLRowEqual(url_info2, info2));
|
|
|
| - // query a nonexistant URL
|
| + // Query a nonexistent URL.
|
| EXPECT_EQ(0, GetRowForURL(GURL("http://news.google.com/"), &info));
|
|
|
| - // Delete all urls in the domain
|
| + // Delete all urls in the domain.
|
| // TODO(acw): test the new url based delete domain
|
| // EXPECT_TRUE(db.DeleteDomain(kDomainID));
|
|
|
| - // Make sure the urls have been properly removed
|
| + // Make sure the urls have been properly removed.
|
| // TODO(acw): commented out because remove no longer works.
|
| // EXPECT_TRUE(db.GetURLInfo(url1, NULL) == NULL);
|
| // EXPECT_TRUE(db.GetURLInfo(url2, NULL) == NULL);
|
|
|