Revision: 7143
Author: [email protected]
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);

-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to