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

Issue 551103003: Fix for List<Future> is not assignable to List. (Closed)

Created:
6 years, 3 months ago by scheglov
Modified:
6 years, 3 months ago
Reviewers:
Paul Berry, danrubel
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fix for List<Future> is not assignable to List. Instead of pushing core.dart file, we should pre-index SDK. R=danrubel@google.com, paulberry@google.com BUG= Committed: https://code.google.com/p/dart/source/detail?r=40038

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -4 lines) Patch
M pkg/analysis_server/test/services/completion/completion_test_util.dart View 3 chunks +12 lines, -4 lines 1 comment Download

Messages

Total messages: 5 (0 generated)
scheglov
6 years, 3 months ago (2014-09-09 00:55:57 UTC) #1
danrubel
lgtm
6 years, 3 months ago (2014-09-09 02:19:34 UTC) #2
Paul Berry
lgtm I assume this is a fix for the issue we discussed yesterday, where some ...
6 years, 3 months ago (2014-09-09 15:28:25 UTC) #3
scheglov
https://codereview.chromium.org/551103003/diff/1/pkg/analysis_server/test/services/completion/completion_test_util.dart File pkg/analysis_server/test/services/completion/completion_test_util.dart (left): https://codereview.chromium.org/551103003/diff/1/pkg/analysis_server/test/services/completion/completion_test_util.dart#oldcode206 pkg/analysis_server/test/services/completion/completion_test_util.dart:206: addResolvedUnit(MockSdk.LIB_CORE.path, MockSdk.LIB_CORE.content); This line violates the Spec by declaring ...
6 years, 3 months ago (2014-09-09 16:24:10 UTC) #4
scheglov
6 years, 3 months ago (2014-09-09 16:24:27 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as 40038 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698