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

Issue 1355393003: JS inline handles -dynamic (Closed)

Created:
5 years, 3 months ago by Leaf
Modified:
5 years, 3 months ago
Reviewers:
vsm, scheglov
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

This adds support for a DDC special case of inline JS. It also addresses an issue from an earlier CL, makes some readability improvements, and eliminates some now-redundant TODOs. BUG= R=scheglov@google.com, vsm@google.com Committed: https://github.com/dart-lang/sdk/commit/ff86501c8587f2369ec05fc73af855e37867728b

Patch Set 1 #

Patch Set 2 : Readability improvements #

Unified diffs Side-by-side diffs Delta from patch set Stats (+36 lines, -15 lines) Patch
M pkg/analyzer/lib/src/generated/static_type_analyzer.dart View 1 7 chunks +20 lines, -15 lines 0 comments Download
M pkg/analyzer/test/generated/resolver_test.dart View 1 chunk +16 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
vsm
lgtm
5 years, 3 months ago (2015-09-22 23:05:50 UTC) #2
Leaf
5 years, 3 months ago (2015-09-22 23:47:57 UTC) #4
scheglov
LGTM
5 years, 3 months ago (2015-09-23 00:03:09 UTC) #5
Leaf
5 years, 3 months ago (2015-09-23 15:58:10 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
ff86501c8587f2369ec05fc73af855e37867728b (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698