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

Issue 10267020: Move all the builtin$ interceptors to a new interceptors library and shorten the names of them. (Closed)

Created:
8 years, 7 months ago by kasperl
Modified:
8 years, 7 months ago
Reviewers:
floitsch
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Move all the builtin$ interceptors to a new interceptors library and shorten the names of them. This re-lands r7187 with a few fixes for VM tests and leg tests. R=floitsch@google.com BUG= TEST= Committed: https://code.google.com/p/dart/source/detail?r=7191

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+700 lines, -654 lines) Patch
M frog/tests/leg/array_static_intercept_test.dart View 1 chunk +3 lines, -3 lines 0 comments Download
M frog/tests/leg/mock_compiler.dart View 3 chunks +18 lines, -8 lines 0 comments Download
M lib/compiler/implementation/compiler.dart View 3 chunks +14 lines, -6 lines 0 comments Download
M lib/compiler/implementation/elements/elements.dart View 1 chunk +11 lines, -0 lines 0 comments Download
A lib/compiler/implementation/lib/interceptors.dart View 1 chunk +636 lines, -0 lines 0 comments Download
M lib/compiler/implementation/lib/js_helper.dart View 12 chunks +1 line, -631 lines 0 comments Download
M lib/compiler/implementation/library_map.dart View 1 chunk +1 line, -0 lines 0 comments Download
M lib/compiler/implementation/ssa/builder.dart View 2 chunks +16 lines, -6 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
kasperl
TBR'ed.
8 years, 7 months ago (2012-05-01 15:19:42 UTC) #1
floitsch
8 years, 7 months ago (2012-05-01 15:38:41 UTC) #2
LGTM.

Powered by Google App Engine
This is Rietveld 408576698