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

Issue 1044463004: Gracefully degrade if no source for completion (Closed)

Created:
5 years, 8 months ago by danrubel
Modified:
5 years, 8 months ago
Reviewers:
scheglov
CC:
reviews_dartlang.org
Target Ref:
refs/remotes/git-svn
Visibility:
Public.

Description

Gracefully degrade if no source for completion BUG=dartbug.com/22617 R=scheglov@google.com Committed: https://code.google.com/p/dart/source/detail?r=44796

Patch Set 1 #

Patch Set 2 : Gracefully degrade if no source for completion #

Total comments: 2

Patch Set 3 : add null check #

Patch Set 4 : merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+110 lines, -28 lines) Patch
M pkg/analysis_server/lib/src/domain_completion.dart View 1 2 1 chunk +3 lines, -0 lines 0 comments Download
M pkg/analysis_server/lib/src/protocol.dart View 1 2 3 1 chunk +8 lines, -0 lines 0 comments Download
M pkg/analysis_server/test/domain_completion_test.dart View 1 1 chunk +5 lines, -0 lines 0 comments Download
M pkg/analysis_server/test/integration/completion/get_suggestions_test.dart View 1 2 chunks +94 lines, -28 lines 0 comments Download

Messages

Total messages: 6 (1 generated)
danrubel
5 years, 8 months ago (2015-03-30 15:39:42 UTC) #2
danrubel
Updated CL. PTAL
5 years, 8 months ago (2015-03-30 21:05:19 UTC) #3
scheglov
lgtm https://codereview.chromium.org/1044463004/diff/20001/pkg/analysis_server/lib/src/domain_completion.dart File pkg/analysis_server/lib/src/domain_completion.dart (right): https://codereview.chromium.org/1044463004/diff/20001/pkg/analysis_server/lib/src/domain_completion.dart#newcode180 pkg/analysis_server/lib/src/domain_completion.dart:180: if (!context.exists(source)) { Actually it is not guaranteed ...
5 years, 8 months ago (2015-03-30 21:10:48 UTC) #4
danrubel
https://codereview.chromium.org/1044463004/diff/20001/pkg/analysis_server/lib/src/domain_completion.dart File pkg/analysis_server/lib/src/domain_completion.dart (right): https://codereview.chromium.org/1044463004/diff/20001/pkg/analysis_server/lib/src/domain_completion.dart#newcode180 pkg/analysis_server/lib/src/domain_completion.dart:180: if (!context.exists(source)) { On 2015/03/30 21:10:48, scheglov wrote: > ...
5 years, 8 months ago (2015-03-30 21:40:48 UTC) #5
danrubel
5 years, 8 months ago (2015-03-30 21:43:14 UTC) #6
Message was sent while issue was closed.
Committed patchset #4 (id:60001) manually as 44796 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698