Make Variable::is_this always return the correct value

src/compiler/ast-graph-builder.cc

Index: src/compiler/ast-graph-builder.cc
diff --git a/src/compiler/ast-graph-builder.cc b/src/compiler/ast-graph-builder.cc
index 7d752a81eb7162069ba5cd4be29315853e3e1ea6..efb065fca09e1ccbf902ce91fcc965c94e00a6cf 100644
--- a/src/compiler/ast-graph-builder.cc
+++ b/src/compiler/ast-graph-builder.cc
@@ -2941,7 +2941,7 @@ void AstGraphBuilder::VisitDelete(UnaryOperation* expr) {
     Variable* variable = expr->expression()->AsVariableProxy()->var();
     // Delete of an unqualified identifier is disallowed in strict mode but

[Michael Starzinger, 2016/08/10 19:45:13]

nit: I know this is not your code, but while you are here, could you drop one of
those two comments that say the same thing twice. Thanks! :)

[Toon Verwaest, 2016/08/11 08:46:25]

Done.

     // "delete this" is allowed.
-    DCHECK(is_sloppy(language_mode()) || variable->HasThisName(isolate()));
+    DCHECK(is_sloppy(language_mode()) || variable->is_this());
     value = BuildVariableDelete(variable, expr->id(),
                                 ast_context()->GetStateCombine());
   } else if (expr->expression()->IsProperty()) {
@@ -3403,7 +3403,7 @@ Node* AstGraphBuilder::BuildVariableDelete(Variable* variable,
     case VariableLocation::LOCAL:
     case VariableLocation::CONTEXT: {
       // Local var, const, or let variable or context variable.
-      return jsgraph()->BooleanConstant(variable->HasThisName(isolate()));
+      return jsgraph()->BooleanConstant(variable->is_this());
     }
     case VariableLocation::LOOKUP: {
       // Dynamic lookup of context variable (anywhere in the chain).