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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: runtime/vm/dart.cc

Issue 1711163002: Remove more things (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 10 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
« runtime/vm/code_observers.h ('K') | « runtime/vm/compiler_stats.cc ('k') | runtime/vm/dart_api_impl.cc » ('j') | runtime/vm/os_linux.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/dart.cc
diff --git a/runtime/vm/dart.cc b/runtime/vm/dart.cc
index 5e563ebf3424c0ccfe39f4a1ec2cc969f0fe6c51..e382fbf5d9e7ec3578863979edc80876e79f2318 100644
--- a/runtime/vm/dart.cc
+++ b/runtime/vm/dart.cc
@@ -107,7 +107,9 @@ const char* Dart::InitOnce(const uint8_t* vm_isolate_snapshot,
PortMap::InitOnce();
FreeListElement::InitOnce();
Api::InitOnce();
+#ifndef PRODUCT
CodeObservers::InitOnce();
+#endif
siva 2016/02/18 23:34:03 Can we move this up so that it is in the same bloc
Cutch 2016/02/22 16:05:44 I've used the new NOT_IN_PRODUCT macro.
if (FLAG_profiler) {
ThreadInterrupter::InitOnce();
Profiler::InitOnce();
@@ -365,7 +367,9 @@ const char* Dart::Cleanup() {
OS::PrintErr("[+%" Pd64 "ms] SHUTDOWN: Deleting code observers\n",
timestamp());
}
+#ifndef PRODUCT
CodeObservers::DeleteAll();
+#endif // !PRODUCT
if (FLAG_support_timeline) {
if (FLAG_trace_shutdown) {
OS::PrintErr("[+%" Pd64 "ms] SHUTDOWN: Shutting down timeline\n",
« runtime/vm/code_observers.h ('K') | « runtime/vm/compiler_stats.cc ('k') | runtime/vm/dart_api_impl.cc » ('j') | runtime/vm/os_linux.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698