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

Unified Diff: runtime/vm/precompiler.cc

Issue 2428353004: Tearing off a top-level variable introduces a synthetic field which is added to the top-level class… (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/precompiler.cc
diff --git a/runtime/vm/precompiler.cc b/runtime/vm/precompiler.cc
index 22348eb43e4b6340e755c19115f1bbbfc7ab70ce..de1f3536e391dbdd84ff30d999390597e95790ae 100644
--- a/runtime/vm/precompiler.cc
+++ b/runtime/vm/precompiler.cc
@@ -1721,8 +1721,7 @@ void Precompiler::DropFields() {
bool top_level = cls.IsTopLevel();
if (top_level) {
name = field.DictionaryName();
- bool removed = lib.RemoveObject(field, name);
- ASSERT(removed);
+ lib.RemoveObject(field, name);
}
dropped_field_count_++;
if (FLAG_trace_precompiler) {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698