499661 - Assertion failure: OBJ_SCOPE(obj)->object == pobj

Status: RESOLVED INVALID

(Core :: JavaScript Engine, defect)

Description

[Mark Banner (:standard8)]

Attachment: Stack & DumpJSStack

When testing send/receive of emails latest Thunderbird 1.9.1 builds I'm seeing:

Assertion failure: OBJ_SCOPE(obj)->object == pobj, at /Users/moztest/comm/main/src/mozilla/js/src/jsobj.cpp:4007

Backtrace and dump attached.

The relevant code is approximately

function nsMsgStatusFeedback()
{
...
}

nsMsgStatusFeedback.prototype =
{

  updateProgress: function() {
    if (this._meteorsSpinning) {
      // In this function, we expect that the maximum for each progress is 100,
      // i.e. we are dealing with percentages. Hence we can combine several
      // processes running at the same time.
      let currentProgress = 0;
      let progressCount = 0;

      // For each activity that is in progress, get its status.

      this._activeProcesses.forEach(function (element) {
          if (element.state ==
              Components.interfaces.nsIActivityProcess.STATE_INPROGRESS &&
              element.percentComplete != -1) {
            currentProgress += element.percentComplete;
            ++progressCount;
          }
        });
...
};

I think the DumpJSStack says it is the ++progressCount line that is being run when JS_Assert is being called.

Link to full source:

http://hg.mozilla.org/comm-central/annotate/6a163fa58973/mail/base/content/mailWindow.js#l331

I wasn't able to form a minimal testcase in a short time.

I would have said this is a regression from bug 498993 however assuming Thunderbird is rebuilding correctly, I backed out that bug and still reproduced the issue.

Comment 1

[Mark Banner (:standard8)]

I guess this could be because I've commented out the assertion I'm getting in bug 497355. Either way, both of these combined make TB debug unusable for me.

Comment 2

[Samuel Sidler (old account; do not CC)]

We won't block on this for 1.9.1.1, but feel free to attach a patch and request approval. (You might need a different owner if Igor is out...)

Comment 3

[Mark Banner (:standard8)]

I have a feeling this is a result of me commenting out the assertion for bug 497355. I haven't seen it since I've applied the patch for that bug, although I haven't done quite as much running.

As no-one else has said they have seen this, I'm going to resolve invalid for now.