Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(52)

Issue 1642973002: SourceFactory interface extraction. (Closed)

Created:
4 years, 10 months ago by pquitslund
Modified:
4 years, 10 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

SourceFactory interface extraction. Prerequisite to accessing resolvers to handle embedder analysis changes. See: https://github.com/dart-lang/sdk/issues/25606 BUG= R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/ce0c3343a726fc392573362a4dc6f580ff396336

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+348 lines, -225 lines) Patch
A pkg/analyzer/lib/src/context/source.dart View 1 chunk +311 lines, -0 lines 4 comments Download
M pkg/analyzer/lib/src/generated/source.dart View 5 chunks +37 lines, -225 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
pquitslund
4 years, 10 months ago (2016-01-28 18:50:06 UTC) #2
Brian Wilkerson
LGTM https://codereview.chromium.org/1642973002/diff/1/pkg/analyzer/lib/src/context/source.dart File pkg/analyzer/lib/src/context/source.dart (right): https://codereview.chromium.org/1642973002/diff/1/pkg/analyzer/lib/src/context/source.dart#newcode1 pkg/analyzer/lib/src/context/source.dart:1: // Copyright (c) 2016, the Dart project authors. ...
4 years, 10 months ago (2016-01-28 18:55:41 UTC) #3
pquitslund
Committed patchset #1 (id:1) manually as ce0c3343a726fc392573362a4dc6f580ff396336 (presubmit successful).
4 years, 10 months ago (2016-01-28 18:57:35 UTC) #5
pquitslund
4 years, 10 months ago (2016-01-28 18:58:20 UTC) #6
Message was sent while issue was closed.
Thanks!

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

https://codereview.chromium.org/1642973002/diff/1/pkg/analyzer/lib/src/contex...
pkg/analyzer/lib/src/context/source.dart:1: // Copyright (c) 2016, the Dart
project authors.  Please see the AUTHORS file
On 2016/01/28 18:55:41, Brian Wilkerson wrote:
> I've been preserving the copyright from the original file when extracting code
> to a new file. Not sure it's important.

Done.

https://codereview.chromium.org/1642973002/diff/1/pkg/analyzer/lib/src/contex...
pkg/analyzer/lib/src/context/source.dart:22: class SourceFactoryImpl implements
SourceFactory {
On 2016/01/28 18:55:41, Brian Wilkerson wrote:
> I'm assuming this was just copied and renamed from the old version of
> SourceFactory. If there are changes I should look at, let me know.

Nope.  _resolvers => resolvers is all.

Powered by Google App Engine
This is Rietveld 408576698