tools/gn: Remove code for forward_dependent_configs_from variable.

tools/gn/target_unittest.cc

 // Copyright (c) 2013 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 "tools/gn/build_settings.h"
 #include "tools/gn/config.h"
 #include "tools/gn/scheduler.h"
 #include "tools/gn/settings.h"
 #include "tools/gn/target.h"
@@ skipping 58 matching lines @@
 
   // Executable target shouldn't get either by depending on shared.
   TestTarget exec(setup, "//foo:exec", Target::EXECUTABLE);
   exec.private_deps().push_back(LabelTargetPair(&shared));
   ASSERT_TRUE(exec.OnResolved(&err));
   EXPECT_EQ(0u, exec.all_libs().size());
   EXPECT_EQ(0u, exec.all_lib_dirs().size());
 }
 
 // Test all_dependent_configs, public_config inheritance, and
-// forward_dependent_configs_from
+// forward_dependent_configs.

[brettw, 2015/10/05 18:06:06]

I think you remove the forward_dependent_configs reference here.

[tfarina, 2015/10/06 03:28:50]

Done.

 TEST(Target, DependentConfigs) {
   TestWithScope setup;
   Err err;
 
   // Set up a dependency chain of a -> b -> c
   TestTarget a(setup, "//foo:a", Target::EXECUTABLE);
   TestTarget b(setup, "//foo:b", Target::STATIC_LIBRARY);
   TestTarget c(setup, "//foo:c", Target::STATIC_LIBRARY);
   a.private_deps().push_back(LabelTargetPair(&b));
   b.private_deps().push_back(LabelTargetPair(&c));
@@ skipping 27 matching lines @@
   // A should have just gotten the "all" dependent config from C.
   ASSERT_EQ(1u, a.configs().size());
   EXPECT_EQ(&all, a.configs()[0].ptr);
   EXPECT_EQ(&all, a.all_dependent_configs()[0].ptr);
 
   // Making an an alternate A and B with B forwarding the direct dependents.
   TestTarget a_fwd(setup, "//foo:a_fwd", Target::EXECUTABLE);
   TestTarget b_fwd(setup, "//foo:b_fwd", Target::STATIC_LIBRARY);
   a_fwd.private_deps().push_back(LabelTargetPair(&b_fwd));
   b_fwd.private_deps().push_back(LabelTargetPair(&c));
   b_fwd.forward_dependent_configs().push_back(LabelTargetPair(&c));

[brettw, 2015/10/05 18:06:06]

Delete this. I'm not sure if any expectations below need to be updated.

[tfarina, 2015/10/06 03:28:50]

Done.

 
   ASSERT_TRUE(b_fwd.OnResolved(&err));
   ASSERT_TRUE(a_fwd.OnResolved(&err));
 
   // A_fwd should now have both configs.
   ASSERT_EQ(2u, a_fwd.configs().size());
   EXPECT_EQ(&all, a_fwd.configs()[0].ptr);
   EXPECT_EQ(&direct, a_fwd.configs()[1].ptr);
   ASSERT_EQ(1u, a_fwd.all_dependent_configs().size());
   EXPECT_EQ(&all, a_fwd.all_dependent_configs()[0].ptr);
@@ skipping 474 matching lines @@
       "\n"
       "From //foo:bar\n"
       "  header: pch.h\n"
       "  source: //pcs.cc\n"
       "\n"
       "From //foo:c2\n"
       "  header: pch2.h\n"
       "  source: //pcs2.cc",
       err.help_text());
 }