Fill CacheEntry.modificationTime for implicit sources.

Fill CacheEntry.modificationTime for implicit sources.

So that _validateCacheConsistency() will do nothing for them.

R=brianwilkerson@google.com
BUG=

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

[Brian Wilkerson, 2015-05-22 18:58:52]

LGTM

https://codereview.chromium.org/1148413004/diff/1/pkg/analyzer/lib/src/contex...
File pkg/analyzer/lib/src/context/context.dart (right):

https://codereview.chromium.org/1148413004/diff/1/pkg/analyzer/lib/src/contex...
pkg/analyzer/lib/src/context/context.dart:683: entry.modificationTime =
getModificationStamp(target);
Unless we explicitly want to not have a modification time for non-Source
targets, this could be:

Source source = target.source;
if (source != null) {
  entry.modificationTime = getModificationStamp(source);
}

[scheglov, 2015-05-22 19:17:30]

https://codereview.chromium.org/1148413004/diff/1/pkg/analyzer/lib/src/contex...
File pkg/analyzer/lib/src/context/context.dart (right):

https://codereview.chromium.org/1148413004/diff/1/pkg/analyzer/lib/src/contex...
pkg/analyzer/lib/src/context/context.dart:683: entry.modificationTime =
getModificationStamp(target);
On 2015/05/22 18:58:52, Brian Wilkerson wrote:
> Unless we explicitly want to not have a modification time for non-Source
> targets, this could be:
> 
> Source source = target.source;
> if (source != null) {
>   entry.modificationTime = getModificationStamp(source);
> }

We use "modificationTime" only "_validateCacheConsistency", and do this only for
sources.
So, it seems that modificationTime for non-sources would not be used, at least
currently.

[scheglov, 2015-05-22 19:18:16]

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