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

Issue 2766173002: Fix class finalization issues in kernel (Closed)

Created:
3 years, 9 months ago by jensj
Modified:
3 years, 9 months ago
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix class finalization issues in kernel Without this we'll get something like ``` ../../runtime/vm/class_finalizer.cc: 2525: error: expected: cls.is_type_finalized() Dumping native stack trace for thread 1b6fc [0x00000000009b6353] dart::Profiler::DumpStackTrace() [0x00000000009b6353] dart::Profiler::DumpStackTrace() [0x0000000000692a21] dart::DynamicAssertionHelper::Fail(char const*, ...) [0x0000000000698420] dart::ClassFinalizer::FinalizeClass(dart::Class const&) [...] ``` when running via kernel with command line option --error_on_bad_override This is caused by finalizing a class having a interface not yet type finalized. This change makes sure types are finalized before the class using them is. Fixes https://github.com/dart-lang/sdk/issues/29123 R=kmillikin@google.com Committed: https://github.com/dart-lang/sdk/commit/242be22300c3cfb99b298651a92237aced3a86fd

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -4 lines) Patch
M runtime/vm/class_finalizer.cc View 1 chunk +9 lines, -4 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
jensj
Probably another solution could be to order the class_array somehow...
3 years, 9 months ago (2017-03-22 10:59:25 UTC) #2
Kevin Millikin (Google)
LGTM.
3 years, 9 months ago (2017-03-22 11:29:50 UTC) #3
jensj
3 years, 9 months ago (2017-03-22 11:32:38 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
242be22300c3cfb99b298651a92237aced3a86fd (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698