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

Issue 2991873002: Store dependency libraries canonical names during limited store. (Closed)

Created:
3 years, 4 months ago by scheglov
Modified:
3 years, 4 months ago
CC:
reviews_dartlang.org, dart-fe-team+reviews_google.com
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+55 lines, -12 lines) Patch
M pkg/front_end/test/src/incremental/kernel_driver_test.dart View 1 chunk +37 lines, -0 lines 0 comments Download
M pkg/kernel/lib/binary/limited_ast_to_binary.dart View 3 chunks +18 lines, -12 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
scheglov
3 years, 4 months ago (2017-07-27 16:16:58 UTC) #1
Brian Wilkerson
I don't think I have the background for this review.
3 years, 4 months ago (2017-07-27 16:53:16 UTC) #2
Siggi Cherem (dart-lang)
lgtm
3 years, 4 months ago (2017-07-27 18:06:27 UTC) #3
scheglov
3 years, 4 months ago (2017-07-27 18:22:11 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
f6038897669b67dfa03893009b21ee0bf97238c2 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698