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

Issue 1177563003: Generate metadata (Closed)

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

Description

Generate metadata Fixes #198 BUG=https://github.com/dart-lang/dev_compiler/issues/198 R=jmesserly@google.com Committed: https://github.com/dart-lang/dev_compiler/commit/99d9a940966a191801589aa52215ba342aff1eae

Patch Set 1 #

Total comments: 6
Unified diffs Side-by-side diffs Delta from patch set Stats (+74 lines, -23 lines) Patch
M lib/runtime/dart/_interceptors.js View 4 chunks +4 lines, -0 lines 2 comments Download
M lib/runtime/dart/_native_typed_data.js View 13 chunks +14 lines, -2 lines 0 comments Download
M lib/runtime/dart/async.js View 1 chunk +1 line, -0 lines 0 comments Download
M lib/runtime/dart_runtime.js View 1 chunk +1 line, -0 lines 0 comments Download
M lib/src/codegen/js_codegen.dart View 12 chunks +54 lines, -21 lines 4 comments Download

Messages

Total messages: 5 (1 generated)
vsm
5 years, 6 months ago (2015-06-09 23:15:42 UTC) #2
Jennifer Messerly
lgtm! https://codereview.chromium.org/1177563003/diff/1/lib/runtime/dart/_interceptors.js File lib/runtime/dart/_interceptors.js (right): https://codereview.chromium.org/1177563003/diff/1/lib/runtime/dart/_interceptors.js#newcode451 lib/runtime/dart/_interceptors.js:451: JSArray[dart.metadata] = () => [dart.const(new _js_helper.JsPeerInterface({name: 'Array'}))]; teach ...
5 years, 6 months ago (2015-06-09 23:29:53 UTC) #3
vsm
https://codereview.chromium.org/1177563003/diff/1/lib/runtime/dart/_interceptors.js File lib/runtime/dart/_interceptors.js (right): https://codereview.chromium.org/1177563003/diff/1/lib/runtime/dart/_interceptors.js#newcode451 lib/runtime/dart/_interceptors.js:451: JSArray[dart.metadata] = () => [dart.const(new _js_helper.JsPeerInterface({name: 'Array'}))]; On 2015/06/09 ...
5 years, 6 months ago (2015-06-10 13:28:27 UTC) #4
vsm
5 years, 6 months ago (2015-06-10 13:28:45 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
99d9a940966a191801589aa52215ba342aff1eae (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698