add property accessor suggestions

pkg/analysis_server/lib/src/services/completion/invocation_computer.dart

Index: pkg/analysis_server/lib/src/services/completion/invocation_computer.dart
diff --git a/pkg/analysis_server/lib/src/services/completion/invocation_computer.dart b/pkg/analysis_server/lib/src/services/completion/invocation_computer.dart
index 798e69d999a3d437ca6e11863071a0b3367449e4..69debf42e0d662a5470585ab7a2d5ac5a2ee903d 100644
--- a/pkg/analysis_server/lib/src/services/completion/invocation_computer.dart
+++ b/pkg/analysis_server/lib/src/services/completion/invocation_computer.dart
@@ -162,6 +162,21 @@ class _InvocationElementVisitor extends GeneralizingElementVisitor<Future<bool>>
   }
 
   @override
+  Future<bool> visitPropertyAccessorElement(PropertyAccessorElement element) {
+    if (element != null) {
+      PropertyInducingElement elemVar = element.variable;
+      if (elemVar != null) {
+        DartType type = elemVar.type;

[Paul Berry, 2014/10/27 22:12:31]

Will this do the right thing if the propagated type is better than the static
type?  E.g.

  var x = "foo";
  x.

should offer completions for String.

Assuming this already works properly, it would be nice to have a test for it.

+        if (type != null) {
+          ClassElementSuggestionBuilder.suggestionsFor(request, type.element);
+        }
+      }
+      return new Future.value(true);
+    }
+    return new Future.value(false);
+  }
+
+  @override
   Future<bool> visitVariableElement(VariableElement element) {
     DartType type = element.type;
     if (type != null) {