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

Issue 2035583004: Fix a few cases were obj and name were used instead of $obj and $name. remove ProfileEntry class as… (Closed)

Created:
4 years, 6 months ago by Jacob
Modified:
4 years, 6 months ago
Reviewers:
priscillalee
CC:
dev-compiler+reviews_dartlang.org, Leaf
Base URL:
git@github.com:dart-lang/dev_compiler.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix a few cases were obj and name were used instead of $obj and $name. remove ProfileEntry class as sadly defining a class in operations.dart appears to have an ordering issue which causes two tests to fail. Committed: https://github.com/dart-lang/dev_compiler/commit/c45d1730ace8495116d21ddf2c20cf8421660a96

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -58 lines) Patch
M lib/runtime/dart_sdk.js View 7 chunks +34 lines, -43 lines 0 comments Download
M tool/input_sdk/private/ddc_runtime/operations.dart View 5 chunks +7 lines, -15 lines 0 comments Download

Messages

Total messages: 4 (2 generated)
Jacob
Tweak to fix a couple failing tests. TBR
4 years, 6 months ago (2016-06-02 20:07:39 UTC) #2
Jacob
4 years, 6 months ago (2016-06-02 20:07:57 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
c45d1730ace8495116d21ddf2c20cf8421660a96 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698