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

Unified Diff: runtime/bin/main.cc

Issue 2760293002: [dart:io] Adds a finalizer to _NativeSocket to avoid socket leaks (Closed)
Patch Set: Address comments Created 3 years, 9 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 | « runtime/bin/eventhandler_win.cc ('k') | runtime/bin/process.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/bin/main.cc
diff --git a/runtime/bin/main.cc b/runtime/bin/main.cc
index 7f294399bb928f8546d1c5fa8e1db84ad6e1171b..10c5605ca37cab3c5cc7af2528d7ecdc411b83fb 100644
--- a/runtime/bin/main.cc
+++ b/runtime/bin/main.cc
@@ -1269,6 +1269,7 @@ bool RunMainIsolate(const char* script_name, CommandLineOptions* dart_options) {
Log::PrintErr("VM cleanup failed: %s\n", error);
free(error);
}
+ Process::ClearAllSignalHandlers();
EventHandler::Stop();
Platform::Exit((exit_code != 0) ? exit_code : kErrorExitCode);
}
@@ -1687,6 +1688,7 @@ void main(int argc, char** argv) {
Log::PrintErr("VM cleanup failed: %s\n", error);
free(error);
}
+ Process::ClearAllSignalHandlers();
EventHandler::Stop();
delete app_snapshot;
« no previous file with comments | « runtime/bin/eventhandler_win.cc ('k') | runtime/bin/process.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698