add optional importUri to completion suggestion

add optional importUri to completion suggestion

BUG=dartbug.com/23175
R=brianwilkerson@google.com

Committed: https://code.google.com/p/dart/source/detail?r=45288

[danrubel, 2015-04-20 14:13:04]

danrubel@google.com changed reviewers:
+ brianwilkerson@google.com

[Brian Wilkerson, 2015-04-20 15:00:23]

LGTM except that we need to bump the version number of the specification

[danrubel, 2015-04-20 15:20:48]

On 2015/04/20 15:00:23, Brian Wilkerson wrote:
> LGTM except that we need to bump the version number of the specification

Bumped. PTAL.

[Brian Wilkerson, 2015-04-20 15:27:10]

https://codereview.chromium.org/1097083002/diff/40001/pkg/analysis_server/too...
File pkg/analysis_server/tool/spec/spec_input.html (right):

https://codereview.chromium.org/1097083002/diff/40001/pkg/analysis_server/too...
pkg/analysis_server/tool/spec/spec_input.html:8: <h1
style="color:#999999">Version <version>1.6.1</version></h1>
I don't know whether that's right or not. Does it make sense to have a "bug fix"
release for a specification? Perhaps a bug fix release should be used for
something like clarifying the wording in the spec. Does the fact that this adds
support for a new field mean that the version should be "1.7.0"? I think that's
how I would expect it to work, but I'm not an expert, so I'll leave it to your
judgement.

[danrubel, 2015-04-20 15:31:39]

On 2015/04/20 15:27:10, Brian Wilkerson wrote:
>
https://codereview.chromium.org/1097083002/diff/40001/pkg/analysis_server/too...
> File pkg/analysis_server/tool/spec/spec_input.html (right):
> 
>
https://codereview.chromium.org/1097083002/diff/40001/pkg/analysis_server/too...
> pkg/analysis_server/tool/spec/spec_input.html:8: <h1
> style="color:#999999">Version <version>1.6.1</version></h1>
> I don't know whether that's right or not. Does it make sense to have a "bug
fix"
> release for a specification? Perhaps a bug fix release should be used for
> something like clarifying the wording in the spec. Does the fact that this
adds
> support for a new field mean that the version should be "1.7.0"? I think
that's
> how I would expect it to work, but I'm not an expert, so I'll leave it to your
> judgement.

I deemed it a bug fix because it is an optional field that should have been
included from the start. Oversight on my part. That said, either is fine by me.
Committing this CL as is. We can bump again if needed.

[danrubel, 2015-04-20 15:33:02]

Committed patchset #4 (id:60001) manually as 45288 (presubmit successful).