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

Issue 1514603002: suggest after import prefix in part files - fixes #25023 (Closed)

Created:
5 years ago by danrubel
Modified:
5 years ago
Reviewers:
scheglov
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

suggest after import prefix in part files - fixes #25023 R=scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/830a37afaeff2e1639c981f0fd98d84ba9db54d5

Patch Set 1 #

Patch Set 2 : resolve merge conflict #

Total comments: 2

Patch Set 3 : merge #

Messages

Total messages: 6 (2 generated)
danrubel
5 years ago (2015-12-09 17:13:47 UTC) #2
scheglov
LGTM https://codereview.chromium.org/1514603002/diff/20001/pkg/analysis_server/lib/src/services/completion/dart/completion_manager.dart File pkg/analysis_server/lib/src/services/completion/dart/completion_manager.dart (right): https://codereview.chromium.org/1514603002/diff/20001/pkg/analysis_server/lib/src/services/completion/dart/completion_manager.dart#newcode106 pkg/analysis_server/lib/src/services/completion/dart/completion_manager.dart:106: List<Source> libraries = context.getLibrariesContaining(source); context.getLibrariesContaining(source) already returns the ...
5 years ago (2015-12-09 19:24:17 UTC) #3
danrubel
https://codereview.chromium.org/1514603002/diff/20001/pkg/analysis_server/lib/src/services/completion/dart/completion_manager.dart File pkg/analysis_server/lib/src/services/completion/dart/completion_manager.dart (right): https://codereview.chromium.org/1514603002/diff/20001/pkg/analysis_server/lib/src/services/completion/dart/completion_manager.dart#newcode106 pkg/analysis_server/lib/src/services/completion/dart/completion_manager.dart:106: List<Source> libraries = context.getLibrariesContaining(source); On 2015/12/09 19:24:17, scheglov wrote: ...
5 years ago (2015-12-09 19:32:41 UTC) #4
danrubel
5 years ago (2015-12-09 19:35:34 UTC) #6
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
830a37afaeff2e1639c981f0fd98d84ba9db54d5 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698