Use server.getContainingContext() in execution domain.

Use server.getContainingContext() in execution domain.

It works even for folders.

R=brianwilkerson@google.com
BUG=

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

[Brian Wilkerson, 2015-05-04 20:06:19]

LGTM

https://codereview.chromium.org/1126613004/diff/1/pkg/analysis_server/lib/src...
File pkg/analysis_server/lib/src/domain_execution.dart (right):

https://codereview.chromium.org/1126613004/diff/1/pkg/analysis_server/lib/src...
pkg/analysis_server/lib/src/domain_execution.dart:99: AnalysisContext context =
server.getContainingContext(path);
Why do we have both methods? They appear to be doing the same thing but in
different ways.

[scheglov, 2015-05-04 20:15:22]

https://codereview.chromium.org/1126613004/diff/1/pkg/analysis_server/lib/src...
File pkg/analysis_server/lib/src/domain_execution.dart (right):

https://codereview.chromium.org/1126613004/diff/1/pkg/analysis_server/lib/src...
pkg/analysis_server/lib/src/domain_execution.dart:99: AnalysisContext context =
server.getContainingContext(path);
On 2015/05/04 20:06:19, Brian Wilkerson wrote:
> Why do we have both methods? They appear to be doing the same thing but in
> different ways.

Mostly for external files.
For example when we get a file from the Pub cache, we want to find it in an
AnalysisContext, even if it is not in an analysis root.

I think we don't need this for the execution domain.

[scheglov, 2015-05-04 20:15:47]

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