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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2983703002: Fix analyzer test after making CompilerContext mandatory (Closed)

Created:
3 years, 5 months ago by Siggi Cherem (dart-lang)
Modified:
3 years, 5 months ago
Reviewers:
ahe, Paul Berry
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix analyzer test after making CompilerContext mandatory (Note: the use of "CompilerCommandLine" will go away in another CL this week) TBR=paulberry@google.com,ahe@google.com Committed: https://github.com/dart-lang/sdk/commit/45d2358fee9a287edab90278fa48e1a4c7b2666f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -19 lines) Patch
M pkg/analyzer/test/src/task/strong/front_end_inference_test.dart View 2 chunks +22 lines, -19 lines 0 comments Download

Messages

Total messages: 3 (1 generated)
Siggi Cherem (dart-lang)
Committed patchset #1 (id:1) manually as 45d2358fee9a287edab90278fa48e1a4c7b2666f (presubmit successful).
3 years, 5 months ago (2017-07-17 21:07:56 UTC) #2
Paul Berry
3 years, 5 months ago (2017-07-17 22:15:35 UTC) #3
Message was sent while issue was closed. 
lgtm

Powered by Google App Engine
This is Rietveld 408576698