Step back and remove more getNode() invocations

compiler/java/com/google/dart/compiler/resolver/Element.java

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 package com.google.dart.compiler.resolver;
 
-import com.google.dart.compiler.ast.DartLabel;
 import com.google.dart.compiler.ast.DartNode;
 import com.google.dart.compiler.ast.Modifiers;
+import com.google.dart.compiler.common.HasSourceInfo;
+import com.google.dart.compiler.common.SourceInfo;
 import com.google.dart.compiler.type.Type;
 
-public interface Element {
+public interface Element extends HasSourceInfo {
   String getOriginalName();
 
   DartNode getNode();
 
-  void setNode(DartLabel node);
-
   String getName();
 
   ElementKind getKind();
 
   Type getType();
 
   boolean isDynamic();
 
   Modifiers getModifiers();
 
   /**
-   * Returns the innermost {@link EnclosingElement} which declares this element.
+   * @return the innermost {@link Element} which declares this {@link Element}.
    */
-  EnclosingElement getEnclosingElement();
+  Element getEnclosingElement();

[zundel, 2012/03/12 19:42:31]

I don't understand this change.  The EnclosingElement says something about the
scope.

[scheglov, 2012/03/13 00:42:25]

I will undo this in the next CL.

+  
+  /**
+   * @return location of the name in the declaration of this {@link Element}.
+   */
+  SourceInfo getNameLocation();

[zundel, 2012/03/12 19:42:31]

We will need to be careful about persisting this field.  There is that static
weak hash map we must make sure is repopulated.

[scheglov, 2012/03/13 00:42:25]

This map is populated lazily.
Basically we will serialize SourceInfo as resource identifier, offset and
length.

 }