Fix const declaration cosmetics and avoid use of computeNode for information analyzer already has. (#1585)

* First version of adding const

* Serve test package docs through grinder

* Almost done, finish test for STUFF

* pubspec update

* Fix label for test package server

* Fix up const-declaration

* dartfmt

* Fix inconsistencies in #1581

* WIP

* rebuild test package docs

Author: jcollins-g

lib/src/model.dart

@@ -13,7 +13,9 @@ import 'package:analyzer/dart/ast/ast.dart'
     show
         AnnotatedNode,
         Declaration,
+        Expression,
         FieldDeclaration,
+        InstanceCreationExpression,
         VariableDeclaration,
         VariableDeclarationList;
 import 'package:analyzer/dart/element/element.dart';
@@ -24,6 +26,7 @@ import 'package:analyzer/source/package_map_resolver.dart';
 import 'package:analyzer/source/sdk_ext.dart';
 // TODO(jcollins-g): Stop using internal analyzer structures somehow.
 import 'package:analyzer/src/context/builder.dart';
+import 'package:analyzer/src/dart/element/element.dart';
 import 'package:analyzer/src/dart/sdk/sdk.dart';
 import 'package:analyzer/src/generated/engine.dart';
 import 'package:analyzer/src/generated/java_io.dart';
@@ -1449,9 +1452,9 @@ class Field extends ModelElement
   bool get isInherited => _isInherited;
 
   @override
-  String get kind => 'property';
+  String get kind => isConst ? 'constant' : 'property';
 
-  String get typeName => "property";
+  String get typeName => kind;
 
   @override
   List<String> get annotations {
@@ -1540,21 +1543,36 @@ abstract class GetterSetterCombo implements ModelElement {
   ModelElement enclosingElement;
   bool get isInherited;
 
-  String _constantValueBase() {
-    if (element.computeNode() != null) {
-      var v = element.computeNode().toSource();
-      if (v == null) return null;
-      var string = v.substring(v.indexOf('=') + 1, v.length).trim();
-      return const HtmlEscape(HtmlEscapeMode.UNKNOWN).convert(string);
+  Expression get constantInitializer =>
+      (element as ConstVariableElement).constantInitializer;
+
+  String linkifyConstantValue(String original) {
+    if (constantInitializer is! InstanceCreationExpression) return original;
+    String constructorName = (constantInitializer as InstanceCreationExpression)
+        .constructorName
+        .toString();
+    Element staticElement =
+        (constantInitializer as InstanceCreationExpression).staticElement;
+    Constructor target = new ModelElement.fromElement(staticElement, package);
+    Class targetClass = target.enclosingElement;
+    // TODO(jcollins-g): this logic really should be integrated into Constructor,
+    // but that's not trivial because of linkedName's usage.
+    if (targetClass.name == target.name) {
+      return original.replaceAll(constructorName, "${target.linkedName}");
     }
-    return null;
+    return original.replaceAll(
+        "${targetClass.name}.${target.name}", "${targetClass.linkedName}.${target.linkedName}");
   }
 
-  String get constantValueBase => _memoizer.memoized(_constantValueBase);
-
-  String get constantValue => constantValueBase;
+  String _constantValueBase() {
+    String result = constantInitializer?.toString() ?? '';
+    return const HtmlEscape(HtmlEscapeMode.UNKNOWN).convert(result);
+  }
 
-  String get constantValueTruncated => truncateString(constantValueBase, 200);
+  String get constantValue => linkifyConstantValue(constantValueBase);
+  String get constantValueTruncated =>
+      linkifyConstantValue(truncateString(constantValueBase, 200));
+  String get constantValueBase => _memoizer.memoized(_constantValueBase);
 
   /// Returns true if both accessors are synthetic.
   bool get hasSyntheticAccessors {
@@ -4632,7 +4650,7 @@ class TopLevelVariable extends ModelElement
   }
 
   @override
-  String get kind => 'top-level property';
+  String get kind => isConst ? 'top-level constant' : 'top-level property';
 
   @override
   Set<String> get features => super.features..addAll(comboFeatures);

lib/templates/_constant.html

@@ -1,6 +1,6 @@
 <dt id="{{htmlId}}" class="constant">
   <span class="name {{#isDeprecated}}deprecated{{/isDeprecated}}">{{{ linkedName }}}</span>
-  <span class="signature">&#8594; {{{ linkedReturnType }}}</span>
+  <span class="signature">&#8594; const {{{ linkedReturnType }}}</span>
 </dt>
 <dd>
   {{{ oneLineDoc }}}

lib/templates/_name_summary.html

@@ -1 +1 @@
-<span class="name {{#isDeprecated}}deprecated{{/isDeprecated}}">{{name}}</span>
+{{#isConst}}const {{/isConst}}<span class="name {{#isDeprecated}}deprecated{{/isDeprecated}}">{{name}}</span>

test/model_test.dart

@@ -809,10 +809,12 @@ void main() {
 
     test('get constants', () {
       expect(Apple.publicConstants, hasLength(1));
+      expect(Apple.publicConstants.first.kind, equals('constant'));
     });
 
     test('get instance fields', () {
       expect(Apple.publicInstanceProperties, hasLength(3));
+      expect(Apple.publicInstanceProperties.first.kind, equals('property'));
     });
 
     test('get inherited properties, including properties of Object', () {
@@ -1911,7 +1913,7 @@ String topLevelFunction(int param1, bool param2, Cool coolBeans,
 
     test('substrings of the constant values type are not linked (#1535)', () {
       expect(aName.constantValue,
-          'const ExtendedShortName("hello there")');
+          'const <a href="ex/ExtendedShortName/ExtendedShortName.html">ExtendedShortName</a>(&quot;hello there&quot;)');
     });
 
     test('constant field values are escaped', () {
@@ -1922,6 +1924,10 @@ String topLevelFunction(int param1, bool param2, Cool coolBeans,
       expect(greenConstant.fullyQualifiedName, 'ex.COLOR_GREEN');
     });
 
+    test('has the correct kind', () {
+      expect(greenConstant.kind, equals('top-level constant'));
+    });
+
     test('has enclosing element', () {
       expect(greenConstant.enclosingElement.name, equals(exLibrary.name));
     });
@@ -1943,8 +1949,8 @@ String topLevelFunction(int param1, bool param2, Cool coolBeans,
           "const &lt;String&gt; [COLOR_GREEN, COLOR_ORANGE, &#39;blue&#39;]");
     });
 
-    test('MY_CAT is not linked', () {
-      expect(cat.constantValue, 'const ConstantCat(&#39;tabby&#39;)');
+    test('MY_CAT is linked', () {
+      expect(cat.constantValue, 'const <a href="ex/ConstantCat/ConstantCat.html">ConstantCat</a>(&#39;tabby&#39;)');
     });
 
     test('exported property', () {

testing/test_package_docs/anonymous_library/doesStuff.html

@@ -54,7 +54,8 @@ <h1>doesStuff function</h1>
 
     <section class="multi-line-signature">
         <span class="returntype">String</span>
-        <span class="name ">doesStuff</span>(<wbr>)
+        <span class="name ">doesStuff</span>
+(<wbr>)
     </section>
 
 

testing/test_package_docs/another_anonymous_lib/greeting.html

@@ -54,7 +54,8 @@ <h1>greeting function</h1>
 
     <section class="multi-line-signature">
         <span class="returntype">String</span>
-        <span class="name ">greeting</span>(<wbr>)
+        <span class="name ">greeting</span>
+(<wbr>)
     </section>
 
 

testing/test_package_docs/css/theOnlyThingInTheLibrary.html

@@ -54,7 +54,8 @@ <h1>theOnlyThingInTheLibrary top-level property</h1>
 
     <section class="multi-line-signature">
       <span class="returntype">String</span>
-      <span class="name ">theOnlyThingInTheLibrary</span>      <div class="features">read / write</div>
+      <span class="name ">theOnlyThingInTheLibrary</span>
+      <div class="features">read / write</div>
     </section>
 
 

testing/test_package_docs/ex/Animal-class.html

@@ -121,7 +121,7 @@ <h2>Constants</h2>
       <dl class="properties">
         <dt id="CAT" class="constant">
           <span class="name ">CAT</span>
-          <span class="signature">&#8594; <a href="ex/Animal-class.html">Animal</a></span>
+          <span class="signature">&#8594; const <a href="ex/Animal-class.html">Animal</a></span>
         </dt>
         <dd>
           <p>Single line docs.</p>
@@ -132,7 +132,7 @@ <h2>Constants</h2>
         </dd>
         <dt id="DOG" class="constant">
           <span class="name ">DOG</span>
-          <span class="signature">&#8594; <a href="ex/Animal-class.html">Animal</a></span>
+          <span class="signature">&#8594; const <a href="ex/Animal-class.html">Animal</a></span>
         </dt>
         <dd>
           <p>Multi line docs.</p>
@@ -144,7 +144,7 @@ <h2>Constants</h2>
         </dd>
         <dt id="HORSE" class="constant">
           <span class="name ">HORSE</span>
-          <span class="signature">&#8594; <a href="ex/Animal-class.html">Animal</a></span>
+          <span class="signature">&#8594; const <a href="ex/Animal-class.html">Animal</a></span>
         </dt>
         <dd>
 
@@ -155,7 +155,7 @@ <h2>Constants</h2>
         </dd>
         <dt id="values" class="constant">
           <span class="name ">values</span>
-          <span class="signature">&#8594; List<span class="signature">&lt;<a href="ex/Animal-class.html">Animal</a>&gt;</span></span>
+          <span class="signature">&#8594; const List<span class="signature">&lt;<a href="ex/Animal-class.html">Animal</a>&gt;</span></span>
         </dt>
         <dd>
           <p>A constant List of the values in this enum, in order of their declaration.</p>

testing/test_package_docs/ex/Animal/hashCode.html

@@ -72,7 +72,8 @@ <h1>hashCode property</h1>
 
         <section class="multi-line-signature">
           <span class="returntype">int</span>
-          <span class="name ">hashCode</span>  <div class="features">inherited</div>
+          <span class="name ">hashCode</span>
+  <div class="features">inherited</div>
 </section>
 
 

testing/test_package_docs/ex/Animal/noSuchMethod.html

@@ -69,7 +69,8 @@ <h1>noSuchMethod method</h1>
 
     <section class="multi-line-signature">
       <span class="returntype">dynamic</span>
-      <span class="name ">noSuchMethod</span>(<wbr><span class="parameter" id="noSuchMethod-param-invocation"><span class="type-annotation">Invocation</span> <span class="parameter-name">invocation</span></span>)
+      <span class="name ">noSuchMethod</span>
+(<wbr><span class="parameter" id="noSuchMethod-param-invocation"><span class="type-annotation">Invocation</span> <span class="parameter-name">invocation</span></span>)
     </section>