Issue 25048. Ignore inner analysis roots which are already managed by outer roots.

Issue 25048. Ignore inner analysis roots which are already managed by outer roots.

R=brianwilkerson@google.com
BUG= https://github.com/dart-lang/sdk/issues/25048

Committed: https://github.com/dart-lang/sdk/commit/5b2ad27b06940ee1005630d431f7b89db5241b38

[Brian Wilkerson, 2015-11-30 22:52:22]

LGTM, with or without the extra test

https://codereview.chromium.org/1487903002/diff/1/pkg/analysis_server/test/co...
File pkg/analysis_server/test/context_manager_test.dart (right):

https://codereview.chromium.org/1487903002/diff/1/pkg/analysis_server/test/co...
pkg/analysis_server/test/context_manager_test.dart:1146: void
test_setRoots_nested_excludedByOuter() {
Might be interesting to try a more deeply nested example. Given a directory
/a/b/c where /a/.analysis_options excludes b, try setting roots to '/a' and
'/a/b/c'.

[scheglov, 2015-11-30 23:01:37]

Description was changed from

==========
Issue 25048. Ignore inner analysis roots which are already managed by outer
roots.

R=brianwilkerson@google.com
BUG= https://github.com/dart-lang/sdk/issues/25048
==========

to

==========
Issue 25048. Ignore inner analysis roots which are already managed by outer
roots.

R=brianwilkerson@google.com
BUG= https://github.com/dart-lang/sdk/issues/25048

Committed:
https://github.com/dart-lang/sdk/commit/5b2ad27b06940ee1005630d431f7b89db5241b38
==========

[scheglov, 2015-11-30 23:01:40]

Committed patchset #2 (id:20001) manually as
5b2ad27b06940ee1005630d431f7b89db5241b38 (presubmit successful).