bookmarks: Rename BookmarkModelTestWithProfile to BookmarkModelTestClient.

components/bookmarks/browser/bookmark_model_unittest.cc

Index: components/bookmarks/browser/bookmark_model_unittest.cc
diff --git a/components/bookmarks/browser/bookmark_model_unittest.cc b/components/bookmarks/browser/bookmark_model_unittest.cc
index 1f71f01a6286d39955bd2e404d3f7452138b2924..c5ff0774f258d30fbae5df3fe7d84239ae9b278c 100644
--- a/components/bookmarks/browser/bookmark_model_unittest.cc
+++ b/components/bookmarks/browser/bookmark_model_unittest.cc
@@ -905,10 +905,9 @@ void PopulateBookmarkNode(TestNode* parent,
   }
 }
 
-// Test class that creates a BookmarkModel with a real history backend.
-class BookmarkModelTestWithProfile : public testing::Test {
+class BookmarkModelTestClient : public testing::Test {

[tfarina, 2015/05/14 00:58:53]

Scott, any suggestion for a better name?

[sky, 2015/05/14 15:13:56]

I find your new name worse. The two functions in this class don't really need to
be member functions. They could be in an anonymous namespace. If you did that,
then there really isn't a need for this class and the tests can go from TEST_F
to TEST. You still need a name though. I would probably go with
BookmarkModelTest2.

[tfarina, 2015/05/14 23:41:18]

Ah, that way is much better! I moved the member functions into the unnamed
namespace.

  public:
-  BookmarkModelTestWithProfile() {}
+  BookmarkModelTestClient() {}
 
  protected:
   // Verifies the contents of the bookmark bar node match the contents of the
@@ -943,12 +942,11 @@ class BookmarkModelTestWithProfile : public testing::Test {
   scoped_ptr<BookmarkModel> model_;
 };
 
-// Creates a set of nodes in the bookmark bar model, then recreates the
-// bookmark bar model which triggers loading from the db and checks the loaded
-// structure to make sure it is what we first created.
-TEST_F(BookmarkModelTestWithProfile, CreateAndRestore) {
+// Creates a set of nodes in the bookmark model, and checks that the loaded
+// structure is what we first created.
+TEST_F(BookmarkModelTestClient, CreateAndRestore) {
   struct TestData {
-    // Structure of the children of the bookmark bar model node.
+    // Structure of the children of the bookmark model node.
     const std::string bbn_contents;
     // Structure of the children of the other node.
     const std::string other_contents;