gn: Allow users to set 'check_includes' in action_foreach().

tools/gn/action_target_generator.cc

Index: tools/gn/action_target_generator.cc
diff --git a/tools/gn/action_target_generator.cc b/tools/gn/action_target_generator.cc
index 09c89e4ca949b5c36e11b883ef665a6b5297351c..933767e8174a819a3cfe79cdb8153ae70fbc5b8f 100644
--- a/tools/gn/action_target_generator.cc
+++ b/tools/gn/action_target_generator.cc
@@ -54,6 +54,9 @@ void ActionTargetGenerator::DoRun() {
   if (!FillDepfile())
     return;
 
+  if (!FillCheckIncludes())
+    return;
+
   if (!CheckOutputs())
     return;
 
@@ -105,6 +108,16 @@ bool ActionTargetGenerator::FillDepfile() {
   return true;
 }
 
+bool ActionTargetGenerator::FillCheckIncludes() {

[brettw, 2015/01/30 23:08:55]

To avoid duplication with the binary target generator, can you move this to
TargetGenerator. Then the different specializations can call it when
appropriate.

[tfarina, 2015/01/30 23:18:29]

Done.

+  const Value* value = scope_->GetValue(variables::kCheckIncludes, true);
+  if (!value)
+    return true;
+  if (!value->VerifyTypeIs(Value::BOOLEAN, err_))
+    return false;
+  target_->set_check_includes(value->boolean_value());
+  return true;
+}
+
 bool ActionTargetGenerator::CheckOutputs() {
   const SubstitutionList& outputs = target_->action_values().outputs();
   if (outputs.list().empty()) {