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

Unified Diff: ceee/ie/broker/executors_manager.cc

Issue 5010001: COM replaced with RPC for firing events in broker.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 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
« no previous file with comments | « ceee/ie/broker/broker_rpc_utils.cc ('k') | ceee/ie/ie.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ceee/ie/broker/executors_manager.cc
===================================================================
--- ceee/ie/broker/executors_manager.cc (revision 66116)
+++ ceee/ie/broker/executors_manager.cc (working copy)
@@ -87,11 +87,6 @@
new_executor_info.thread_handle = thread_handle;
} // End of lock.
- if (map_was_empty) {
- // We go from empty to not empty,
- // so lock the module to make sure we stay alive.
- ceee_module_util::LockModule();
- }
return S_OK;
}
@@ -147,13 +142,6 @@
new_executor_info.thread_handle = thread_handle;
} // End of lock.
- if (map_was_empty) {
- // We go from empty to not empty,
- // so lock the module to make sure we stay alive.
- ceee_module_util::LockModule();
- }
-
- // Update the list of handles that our thread is waiting on.
BOOL success = ::SetEvent(update_threads_list_gate_);
DCHECK(success);
return S_OK;
@@ -266,11 +254,6 @@
map_is_empty = executors_.empty();
} // End of lock.
- if (map_is_empty) {
- // We go from not empty to empty,
- // so unlock the module it can leave in peace.
- ceee_module_util::UnlockModule();
- }
return S_OK;
}
@@ -285,7 +268,6 @@
if (!executors_.empty()) {
// TODO(mad@chromium.org): Can this happen???
NOTREACHED();
- ceee_module_util::UnlockModule();
}
executors_.clear();
« no previous file with comments | « ceee/ie/broker/broker_rpc_utils.cc ('k') | ceee/ie/ie.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698