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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/test/base/layout_test_http_server.cc

Issue 8667006: Extract similar code into SetJobObjectAsKillOnJobClose() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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
« base/process_util_win.cc ('K') | « chrome/common/chrome_content_client.cc ('k') | chrome/test/ui/ui_test_suite.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/base/layout_test_http_server.cc
diff --git a/chrome/test/base/layout_test_http_server.cc b/chrome/test/base/layout_test_http_server.cc
index 7ae0891d9b2071498b92f6217d642f2900edc432..1ee92e038037873871f199270e86654d062549af 100644
--- a/chrome/test/base/layout_test_http_server.cc
+++ b/chrome/test/base/layout_test_http_server.cc
@@ -87,11 +87,7 @@ bool LayoutTestHttpServer::Start() {
return false;
}
- JOBOBJECT_EXTENDED_LIMIT_INFORMATION limit_info = {0};
- limit_info.BasicLimitInformation.LimitFlags =
- JOB_OBJECT_LIMIT_KILL_ON_JOB_CLOSE;
- if (0 == SetInformationJobObject(job_handle_.Get(),
- JobObjectExtendedLimitInformation, &limit_info, sizeof(limit_info))) {
+ if (!SetJobObjectAsKillOnJobClose(job_handle_.Get())) {
LOG(ERROR) << "Could not SetInformationJobObject.";
return false;
}
@@ -136,4 +132,3 @@ bool LayoutTestHttpServer::Stop() {
return stopped;
}
-
« base/process_util_win.cc ('K') | « chrome/common/chrome_content_client.cc ('k') | chrome/test/ui/ui_test_suite.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698