GN: fix datadeps visibility check

tools/gn/target.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 "tools/gn/target.h"
 
 #include "base/bind.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "tools/gn/config_values_extractors.h"
@@ skipping 344 matching lines @@
   // (expanded groups) will have a null origin. We don't want to check
   // visibility for these, since the point of a group would often be to
   // forward visibility.
   for (size_t i = 0; i < deps_.size(); i++) {
     if (deps_[i].origin &&
         !Visibility::CheckItemVisibility(this, deps_[i].ptr, err))
       return false;
   }
 
   for (size_t i = 0; i < datadeps_.size(); i++) {
-    if (deps_[i].origin &&
+    if (datadeps_[i].origin &&
         !Visibility::CheckItemVisibility(this, datadeps_[i].ptr, err))
       return false;
   }
 
   return true;
 }
 
 bool Target::CheckTestonly(Err* err) const {
   // If there current target is marked testonly, it can include both testonly
   // and non-testonly targets, so there's nothing to check.
@@ skipping 10 matching lines @@
 
   for (size_t i = 0; i < datadeps_.size(); i++) {
     if (datadeps_[i].ptr->testonly()) {
       *err = MakeTestOnlyError(this, datadeps_[i].ptr);
       return false;
     }
   }
 
   return true;
 }