Count data deps even if it was already a regular dep

tools/gn/runtime_deps_unittest.cc

 // Copyright 2015 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 <algorithm>
 
 #include "testing/gtest/include/gtest/gtest.h"
 #include "tools/gn/runtime_deps.h"
 #include "tools/gn/target.h"
 #include "tools/gn/test_with_scope.h"
@@ skipping 222 matching lines @@
   EXPECT_TRUE(std::find(result.begin(), result.end(),
                         MakePair("../../dep_copy/data/", &dep_copy)) !=
               result.end()) << GetVectorDescription(result);
 
   // Explicitly asking for the runtime deps of an action target only includes
   // the data and not all outputs.
   result = ComputeRuntimeDeps(&dep);
   ASSERT_EQ(1u, result.size());
   EXPECT_TRUE(MakePair("../../dep.data", &dep) == result[0]);
 }
+
+// Tests that a dependency duplicated in regular and data deps is processed
+// as a data dep.
+TEST(RuntimeDeps, Dupe) {
+  TestWithScope setup;
+  Err err;
+
+  Target action(setup.settings(), Label(SourceDir("//"), "action"));
+  InitTargetWithType(setup, &action, Target::ACTION);
+  action.action_values().outputs() =
+      SubstitutionList::MakeForTest("//action.output");
+  ASSERT_TRUE(action.OnResolved(&err));
+
+  Target target(setup.settings(), Label(SourceDir("//"), "foo"));
+  InitTargetWithType(setup, &target, Target::EXECUTABLE);
+  target.private_deps().push_back(LabelTargetPair(&action));
+  target.data_deps().push_back(LabelTargetPair(&action));
+  ASSERT_TRUE(target.OnResolved(&err));
+
+  // The results should be the executable and the copy output.
+  std::vector<std::pair<OutputFile, const Target*>> result =
+      ComputeRuntimeDeps(&target);
+  EXPECT_TRUE(std::find(result.begin(), result.end(),
+                        MakePair("../../action.output", &action)) !=
+              result.end()) << GetVectorDescription(result);
+}