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

Issue 2617433003: Set ContentCache for created AnalysisContext(s) to control its view of the world. (Closed)

Created:
3 years, 11 months ago by scheglov
Modified:
3 years, 11 months ago
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Set ContentCache for created AnalysisContext(s) to control its view of the world. This should help to avoid inconsistency exceptions during Quick Fixes, when we attempt to match element models created for a source with overlay, and AST that AnalysisContext created for the content without overlay. R=paulberry@google.com, brianwilkerson@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/305d0dff8934efbbc120250b9db4397fba0b38c1

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+61 lines, -15 lines) Patch
M pkg/analyzer/lib/src/dart/analysis/driver.dart View 4 chunks +41 lines, -15 lines 1 comment Download
M pkg/analyzer/lib/src/generated/source.dart View 1 chunk +3 lines, -0 lines 0 comments Download
M pkg/analyzer/test/src/dart/analysis/driver_test.dart View 1 chunk +17 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
scheglov
3 years, 11 months ago (2017-01-04 20:46:01 UTC) #1
Paul Berry
lgtm https://codereview.chromium.org/2617433003/diff/1/pkg/analyzer/lib/src/dart/analysis/driver.dart File pkg/analyzer/lib/src/dart/analysis/driver.dart (right): https://codereview.chromium.org/2617433003/diff/1/pkg/analyzer/lib/src/dart/analysis/driver.dart#newcode1545 pkg/analyzer/lib/src/dart/analysis/driver.dart:1545: return getExists(source) ? 0 : -1; Since the ...
3 years, 11 months ago (2017-01-04 20:50:50 UTC) #2
scheglov
Committed patchset #1 (id:1) manually as 305d0dff8934efbbc120250b9db4397fba0b38c1 (presubmit successful).
3 years, 11 months ago (2017-01-04 20:55:53 UTC) #4
Brian Wilkerson
3 years, 11 months ago (2017-01-04 20:59:36 UTC) #5
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698