Extract DependencyGraph from ProfileDependencyManager.

chrome/browser/profiles/profile_dependency_manager_unittest.cc

 // Copyright (c) 2012 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.
 
 #include "testing/gtest/include/gtest/gtest.h"
 
 #include "chrome/browser/profiles/profile_dependency_manager.h"
 #include "chrome/browser/profiles/profile_keyed_service_factory.h"
 
 class ProfileDependencyManagerUnittests : public ::testing::Test {
  protected:
-  virtual ~ProfileDependencyManagerUnittests() {
-    EXPECT_TRUE(dependency_manager_.all_components_.empty());
-    EXPECT_TRUE(dependency_manager_.edges_.empty());
-    EXPECT_TRUE(dependency_manager_.destruction_order_.empty());
-  }
-
   // To get around class access:
   void DependOn(ProfileKeyedServiceFactory* child,
                 ProfileKeyedServiceFactory* parent) {
     child->DependsOn(parent);
   }
 
   ProfileDependencyManager* manager() { return &dependency_manager_; }
 
   std::vector<std::string>* shutdown_order() { return &shutdown_order_; }
 
@@ skipping 61 matching lines @@
 
   manager()->DestroyProfileServices(NULL);
 
   ASSERT_EQ(3U, shutdown_order()->size());
   EXPECT_STREQ("child2", (*shutdown_order())[0].c_str());
   EXPECT_STREQ("child1", (*shutdown_order())[1].c_str());
   EXPECT_STREQ("parent", (*shutdown_order())[2].c_str());
 }
 
 // Tests an M configuration
 TEST_F(ProfileDependencyManagerUnittests, MConfiguration) {

[Jói, 2013/04/16 08:36:01]

Just to check: Is it intentional to leave the same set of tests in this file as
well?

[Paweł Hajdan Jr., 2013/04/16 16:44:57]

Yes, I think if the tests are there I'd rather keep them - note that
dependency_graph would move to a different place, while
profile_dependency_manager would stay here.

   TestService parent1("parent1", shutdown_order(), manager());
   TestService parent2("parent2", shutdown_order(), manager());
 
   TestService child_of_1("child_of_1", shutdown_order(), manager());
   DependOn(&child_of_1, &parent1);
 
   TestService child_of_12("child_of_12", shutdown_order(), manager());
   DependOn(&child_of_12, &parent1);
   DependOn(&child_of_12, &parent2);
 
@@ skipping 60 matching lines @@
   manager()->DestroyProfileServices(NULL);
 
   ASSERT_EQ(6U, shutdown_order()->size());
   EXPECT_STREQ("bottom", (*shutdown_order())[0].c_str());
   EXPECT_STREQ("specialized_service", (*shutdown_order())[1].c_str());
   EXPECT_STREQ("other_intermediary", (*shutdown_order())[2].c_str());
   EXPECT_STREQ("intermediary_service", (*shutdown_order())[3].c_str());
   EXPECT_STREQ("other_root", (*shutdown_order())[4].c_str());
   EXPECT_STREQ("everything_depends_on_me", (*shutdown_order())[5].c_str());
 }