Flush AST results for source outside of the analysis roots.

Flush AST results for source outside of the analysis roots.

R=brianwilkerson@google.com
BUG=

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

[Brian Wilkerson, 2016-08-23 19:56:32]

https://codereview.chromium.org/2271803002/diff/1/pkg/analysis_server/lib/src...
File pkg/analysis_server/lib/src/operation/operation_analysis.dart (right):

https://codereview.chromium.org/2271803002/diff/1/pkg/analysis_server/lib/src...
pkg/analysis_server/lib/src/operation/operation_analysis.dart:402:
context.analysisCache.flush((target, result) {
This won't work well when we start sharing more analysis between contexts. My
hope is that we can take sharing far enough that when a user opens two packages
A and B and has a dependency override for B to reference A directly that we will
share the results of analyzing A.

If we do get to that point, and if we flush from B, then all of the results of
analyzing A will be outside of B and will get flushed. Perhaps we want to limit
the flushing to the context's private partition.

[Brian Wilkerson, 2016-08-23 20:03:01]

After our discussion, lgtm as is

[scheglov, 2016-08-23 21:53:56]

Description was changed from

==========
Flush AST results for source outside of the analysis roots.

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

to

==========
Flush AST results for source outside of the analysis roots.

R=brianwilkerson@google.com
BUG=

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

[scheglov, 2016-08-23 21:53:58]

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