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

Issue 2031293002: Don't crash when tree shaking is disabled (Closed)

Created:
4 years, 6 months ago by Johnni Winther
Modified:
4 years, 6 months ago
CC:
reviews_dartlang.org
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Don't crash when tree shaking is disabled When tree shaking is disabled, type checking crashed on member lookup into serialized classes. Also classes are now ensured to be resolved when computing lookup. R=sigmund@google.com Committed: https://github.com/dart-lang/sdk/commit/153c36ce3994f46913eea77346e19cb2f90192db

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -0 lines) Patch
M pkg/compiler/lib/src/resolution/class_members.dart View 2 chunks +6 lines, -0 lines 0 comments Download
M tests/compiler/dart2js/serialization/model_test.dart View 1 chunk +4 lines, -0 lines 0 comments Download
M tests/compiler/dart2js/serialization/test_data.dart View 1 chunk +10 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
Johnni Winther
4 years, 6 months ago (2016-06-03 12:48:13 UTC) #2
Siggi Cherem (dart-lang)
lgtm
4 years, 6 months ago (2016-06-03 17:59:34 UTC) #3
Johnni Winther
4 years, 6 months ago (2016-06-06 09:08:10 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
153c36ce3994f46913eea77346e19cb2f90192db (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698