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

Unified Diff: ipc/ipc_sync_channel.cc

Issue 10854092: Add moar logging for "Couldn't load plug-in" errors. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review Created 8 years, 4 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 | « content/common/np_channel_base.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ipc/ipc_sync_channel.cc
diff --git a/ipc/ipc_sync_channel.cc b/ipc/ipc_sync_channel.cc
index 9a897b534bd333a1c8714794df615964000bb0ca..682286f7a57cb239b43397de701a5095ca477fa6 100644
--- a/ipc/ipc_sync_channel.cc
+++ b/ipc/ipc_sync_channel.cc
@@ -301,9 +301,15 @@ bool SyncChannel::SyncContext::TryToUnblockListener(const Message* msg) {
return false;
}
+ // TODO(bauerb): Remove logging once investigation of http://crbug.com/141055
+ // has finished.
if (!msg->is_reply_error()) {
- deserializers_.back().send_result = deserializers_.back().deserializer->
+ bool send_result = deserializers_.back().deserializer->
SerializeOutputParameters(*msg);
+ deserializers_.back().send_result = send_result;
+ LOG_IF(ERROR, !send_result) << "Couldn't deserialize reply message";
+ } else {
+ LOG(ERROR) << "Received error reply";
}
deserializers_.back().done_event->Signal();
@@ -357,6 +363,7 @@ void SyncChannel::SyncContext::OnChannelClosed() {
void SyncChannel::SyncContext::OnSendTimeout(int message_id) {
base::AutoLock auto_lock(deserializers_lock_);
PendingSyncMessageQueue::iterator iter;
+ LOG(ERROR) << "Send timeout";
for (iter = deserializers_.begin(); iter != deserializers_.end(); iter++) {
if (iter->id == message_id) {
iter->done_event->Signal();
@@ -368,6 +375,7 @@ void SyncChannel::SyncContext::OnSendTimeout(int message_id) {
void SyncChannel::SyncContext::CancelPendingSends() {
base::AutoLock auto_lock(deserializers_lock_);
PendingSyncMessageQueue::iterator iter;
+ LOG(ERROR) << "Canceling pending sends";
for (iter = deserializers_.begin(); iter != deserializers_.end(); iter++)
iter->done_event->Signal();
}
@@ -427,6 +435,7 @@ bool SyncChannel::SendWithTimeout(Message* message, int timeout_ms) {
// *this* might get deleted in WaitForReply.
scoped_refptr<SyncContext> context(sync_context());
if (context->shutdown_event()->IsSignaled()) {
+ LOG(ERROR) << "shutdown event is signaled";
delete message;
return false;
}
« no previous file with comments | « content/common/np_channel_base.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698