Start using the new index in Analysis Server.

Start using the new index in Analysis Server.

R=brianwilkerson@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/76f4578af88ef052bd8d644af3651a132b7f8ca7

[Brian Wilkerson, 2016-03-11 21:50:16]

LGTM; your call on all the comments

https://codereview.chromium.org/1786013004/diff/1/pkg/analysis_server/test/se...
File pkg/analysis_server/test/search/element_references_test.dart (right):

https://codereview.chromium.org/1786013004/diff/1/pkg/analysis_server/test/se...
pkg/analysis_server/test/search/element_references_test.dart:742:
fail_potential_method_definedInSubclass() async {
Should we create an issue to track this?

https://codereview.chromium.org/1786013004/diff/1/pkg/analysis_server/test/se...
File pkg/analysis_server/test/services/refactoring/move_file_test.dart (right):

https://codereview.chromium.org/1786013004/diff/1/pkg/analysis_server/test/se...
pkg/analysis_server/test/services/refactoring/move_file_test.dart:32:
fail_file_importedLibrary_package() async {
Should we create an issue for this?

https://codereview.chromium.org/1786013004/diff/1/pkg/analysis_server/test/se...
File pkg/analysis_server/test/services/refactoring/rename_class_member_test.dart
(left):

https://codereview.chromium.org/1786013004/diff/1/pkg/analysis_server/test/se...
pkg/analysis_server/test/services/refactoring/rename_class_member_test.dart:575:
class E {
Not sure why this was removed. I assume it's original purpose was to ensure that
we didn't rename things we shouldn't, and I don't think the semantic changes
effect that.

[scheglov, 2016-03-11 22:02:41]

Description was changed from

==========
Start using the new index in Analysis Server.

R=brianwilkerson@google.com
BUG=
==========

to

==========
Start using the new index in Analysis Server.

R=brianwilkerson@google.com
BUG=

Committed:
https://github.com/dart-lang/sdk/commit/76f4578af88ef052bd8d644af3651a132b7f8ca7
==========

[scheglov, 2016-03-11 22:02:42]

Committed patchset #1 (id:1) manually as
76f4578af88ef052bd8d644af3651a132b7f8ca7 (presubmit successful).

[scheglov, 2016-03-11 22:02:50]

https://codereview.chromium.org/1786013004/diff/1/pkg/analysis_server/test/se...
File pkg/analysis_server/test/services/refactoring/rename_class_member_test.dart
(left):

https://codereview.chromium.org/1786013004/diff/1/pkg/analysis_server/test/se...
pkg/analysis_server/test/services/refactoring/rename_class_member_test.dart:575:
class E {
On 2016/03/11 21:50:16, Brian Wilkerson wrote:
> Not sure why this was removed. I assume it's original purpose was to ensure
that
> we didn't rename things we shouldn't, and I don't think the semantic changes
> effect that.

Thank you for catching this.
It should have been reverted before sending this CL.
Done.