Support explicit pools in actions

tools/gn/action_target_generator.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/action_target_generator.h"
 
 #include "tools/gn/build_settings.h"
 #include "tools/gn/err.h"
 #include "tools/gn/filesystem_utils.h"
+#include "tools/gn/functions.h"
 #include "tools/gn/parse_tree.h"
 #include "tools/gn/scope.h"
 #include "tools/gn/value.h"
 #include "tools/gn/value_extractors.h"
 #include "tools/gn/variables.h"
 
 ActionTargetGenerator::ActionTargetGenerator(
     Target* target,
     Scope* scope,
     const FunctionCallNode* function_call,
@@ skipping 30 matching lines @@
 
   if (!FillResponseFileContents())
     return;
 
   if (!FillOutputs(output_type_ == Target::ACTION_FOREACH))
     return;
 
   if (!FillDepfile())
     return;
 
-  if (!FillConsole())
+  if (!FillPool())
     return;
 
   if (!FillCheckIncludes())
     return;
 
   if (!CheckOutputs())
     return;
 
   // Action outputs don't depend on the current toolchain so we can skip adding
   // that dependency.
@@ skipping 82 matching lines @@
   SubstitutionPattern depfile;
   if (!depfile.Parse(*value, err_))
     return false;
   if (!EnsureSubstitutionIsInOutputDir(depfile, *value))
     return false;
 
   target_->action_values().set_depfile(depfile);
   return true;
 }
 
-bool ActionTargetGenerator::FillConsole() {
-  const Value* value = scope_->GetValue(variables::kConsole, true);
+bool ActionTargetGenerator::FillPool() {
+  const Value* value = scope_->GetValue(variables::kPool, true);
   if (!value)
     return true;
-  if (!value->VerifyTypeIs(Value::BOOLEAN, err_))
+
+  Label label = Label::Resolve(scope_->GetSourceDir(),
+                               ToolchainLabelForScope(scope_), *value, err_);
+  if (err_->has_error())
     return false;
-  target_->action_values().set_console(value->boolean_value());
+
+  LabelPtrPair<Pool> pair(label);
+  pair.origin = target_->defined_from();
+
+  target_->action_values().set_pool(std::move(pair));
   return true;
 }
 
 bool ActionTargetGenerator::CheckOutputs() {
   const SubstitutionList& outputs = target_->action_values().outputs();
   if (outputs.list().empty()) {
     *err_ = Err(function_call_, "Action has no outputs.",
         "If you have no outputs, the build system can not tell when your\n"
         "script needs to be run.");
     return false;
@@ skipping 13 matching lines @@
       *err_ = Err(function_call_,
           "action_foreach should have a pattern in the output.",
           "An action_foreach target should have a source expansion pattern in\n"
           "it to map source file to unique output file name. Otherwise, the\n"
           "build system can't determine when your script needs to be run.");
       return false;
     }
   }
   return true;
 }