[google-web-toolkit] r7143 committed - Improve debug logging to help track down crash....

[codesite...@google.com]

Revision: 7143
Author: j...@google.com
Date: Mon Nov 23 15:38:51 2009
Log: Improve debug logging to help track down crash.

Patch by: jat
Review by: scottb (TBR)

http://code.google.com/p/google-web-toolkit/source/detail?r=7143

Modified:
/trunk/plugins/xpcom/ExternalWrapper.cpp
/trunk/plugins/xpcom/FFSessionHandler.cpp

=======================================
--- /trunk/plugins/xpcom/ExternalWrapper.cpp Tue Nov 10 14:14:58 2009
+++ /trunk/plugins/xpcom/ExternalWrapper.cpp Mon Nov 23 15:38:51 2009
@@ -45,8 +45,8 @@
NS_IMPL_ISUPPORTS2_CI(ExternalWrapper, IOOPHM, nsISecurityCheckedComponent)

ExternalWrapper::ExternalWrapper() {
- Debug::log(Debug::Spam) << "ExternalWrapper::ExternalWrapper()"
- << Debug::flush;
+ Debug::log(Debug::Debugging) << "ExternalWrapper::ExternalWrapper(this="
+ << this << ")" << Debug::flush;
preferences = new Preferences();
windowWatcher = do_GetService(NS_WINDOWWATCHER_CONTRACTID);
if (!windowWatcher) {
@@ -57,7 +57,8 @@
}

ExternalWrapper::~ExternalWrapper() {
- Debug::log(Debug::Spam) << "ExternalWrapper::~ExternalWrapper" <<
Debug::flush;
+ Debug::log(Debug::Debugging) << "ExternalWrapper::~ExternalWrapper(this="
+ << this << ")" << Debug::flush;
}

// define the CID for nsIHttpProtocolHandler
=======================================
--- /trunk/plugins/xpcom/FFSessionHandler.cpp Mon Nov 23 13:18:40 2009
+++ /trunk/plugins/xpcom/FFSessionHandler.cpp Mon Nov 23 15:38:51 2009
@@ -57,6 +57,8 @@
FFSessionHandler::FFSessionHandler(HostChannel* channel)
: SessionData(channel, this, getJSContext()), jsObjectId(0),
jsObjectsById(NULL), stringObjectClass(NULL) {
+ Debug::log(Debug::Debugging)
<< "FFSessionHandler::FFSessionHandler(this="
+ << this << ")" << Debug::flush;
// TODO(jat): is there a way to avoid calling this twice, without keeping
// JSContext in an instance field?
JSContext* ctx = getJSContext();
@@ -118,7 +120,8 @@
}

FFSessionHandler::~FFSessionHandler(void) {
- Debug::log(Debug::Debugging) << "FFSessionHandler::~FFSessionHandler" <<
Debug::flush;
+ Debug::log(Debug::Debugging)
<< "FFSessionHandler::~FFSessionHandler(this="
+ << this << ")" << Debug::flush;
disconnect();
if (runtime) {
JS_RemoveRootRT(runtime, &jsObjectsById);