Track usage of "this" and "arguments" in Scope

src/ast-value-factory.h

Index: src/ast-value-factory.h
diff --git a/src/ast-value-factory.h b/src/ast-value-factory.h
index bb93d10cd25c2369a418d728c9c9e1ff56541668..4804896c1b6df7230e6a8bafbb7c0cb934cf7af6 100644
--- a/src/ast-value-factory.h
+++ b/src/ast-value-factory.h
@@ -88,6 +88,8 @@ class AstRawString : public AstString {
     return *c;
   }
 
+  bool IsArguments() const { return IsOneByteEqualTo("arguments"); }

[marja, 2014/08/11 08:51:39]

This doesn't look correct. The AstStrings are unique, so you should be able to
compare the AstString* to AstValueFactory::arguments_string().

Is the problem that where this is used, there's no AstValueFactory? If so, that
should probably be fixed... I mean, surely the AstValueFactory is *somewhere*
since we managed to create the AstString...

+
   // For storing AstRawStrings in a hash map.
   uint32_t hash() const {
     return hash_;