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

Issue 1921503007: simplify emitting dart.bind for native types (Closed)

Created:
4 years, 7 months ago by Jennifer Messerly
Modified:
4 years, 7 months ago
Reviewers:
vsm, Harry Terkelsen
CC:
dev-compiler+reviews_dartlang.org
Base URL:
git@github.com:dart-lang/dev_compiler.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : merged #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -17 lines) Patch
M lib/runtime/dart_sdk.js View 1 1 chunk +2 lines, -3 lines 0 comments Download
M lib/src/compiler/code_generator.dart View 1 2 3 chunks +15 lines, -11 lines 0 comments Download
M test/codegen/expect/language-all.js View 1 Binary file 0 comments Download
M tool/input_sdk/private/ddc_runtime/classes.dart View 1 1 chunk +2 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
Jennifer Messerly
My main goal here was to fix _emitFunctionTypeParts in _emitAccess. Also tried to improve the ...
4 years, 7 months ago (2016-04-27 16:46:17 UTC) #2
Harry Terkelsen
lgtm
4 years, 7 months ago (2016-04-27 16:53:20 UTC) #3
Jennifer Messerly
4 years, 7 months ago (2016-04-27 16:59:40 UTC) #5
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
5e938fd69a9d605bf2d6f37b45c838aab2ccd152 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698