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

Issue 2782913003: Revert "Remove definitions from Kernel canonical names." (Closed)

Created:
3 years, 8 months ago by zra
Modified:
3 years, 8 months ago
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Revert "Remove definitions from Kernel canonical names." This reverts commit ed77783cd32d55fdad61bf9bc749030847ba9384. BUG= R=fschneider@google.com Committed: https://github.com/dart-lang/sdk/commit/a7d1d99389924e930be4d033559aa678084ce3aa

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+352 lines, -409 lines) Patch
M pkg/kernel/lib/ast.dart View 1 chunk +0 lines, -1 line 0 comments Download
M pkg/kernel/lib/canonical_name.dart View 3 chunks +1 line, -7 lines 0 comments Download
M runtime/vm/kernel.h View 27 chunks +89 lines, -46 lines 0 comments Download
M runtime/vm/kernel.cc View 18 chunks +69 lines, -133 lines 0 comments Download
M runtime/vm/kernel_binary.cc View 7 chunks +24 lines, -6 lines 0 comments Download
M runtime/vm/kernel_reader.cc View 7 chunks +13 lines, -18 lines 0 comments Download
M runtime/vm/kernel_to_il.h View 5 chunks +16 lines, -32 lines 0 comments Download
M runtime/vm/kernel_to_il.cc View 27 chunks +140 lines, -166 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
zra
3 years, 8 months ago (2017-03-29 18:20:52 UTC) #2
Florian Schneider
lgtm
3 years, 8 months ago (2017-03-29 18:22:49 UTC) #3
zra
3 years, 8 months ago (2017-03-29 18:25:22 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
a7d1d99389924e930be4d033559aa678084ce3aa (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698