Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(12)

Unified Diff: tools/gn/scope_per_file_provider.h

Issue 226223006: Template invocation fixes in GN (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/gn/scope_per_file_provider.h
diff --git a/tools/gn/scope_per_file_provider.h b/tools/gn/scope_per_file_provider.h
index 1021c276133c80102bfda7e36e2245f12d1ef1bb..2ef84cc8493864bb0553fcd016de4f58a7d99371 100644
--- a/tools/gn/scope_per_file_provider.h
+++ b/tools/gn/scope_per_file_provider.h
@@ -14,7 +14,10 @@
// variable support.
class ScopePerFileProvider : public Scope::ProgrammaticProvider {
public:
- ScopePerFileProvider(Scope* scope);
+ // allow_target_vars allows the target-related variables to get resolved.
+ // When allow_target_vars is unset, the target-related values will be
+ // undefined to GN script.
+ ScopePerFileProvider(Scope* scope, bool allow_target_vars);
virtual ~ScopePerFileProvider();
// ProgrammaticProvider implementation.
@@ -31,6 +34,8 @@ class ScopePerFileProvider : public Scope::ProgrammaticProvider {
const Value* GetTargetGenDir();
const Value* GetTargetOutDir();
+ bool allow_target_vars_;
+
// All values are lazily created.
scoped_ptr<Value> current_toolchain_;
scoped_ptr<Value> default_toolchain_;
« base/BUILD.gn ('K') | « tools/gn/loader.cc ('k') | tools/gn/scope_per_file_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698