Sorry for top posting.

Mea Culpa:

I was looking at the wrong file as Boris correctly pointed out.

These two bugs were seen
in the timeout test case mentioned first in
https://bugzilla.mozilla.org/show_bug.cgi?id=1081763
Bug 1081763 - C-C TB: gContextMenu is undefined during test that
eventually times out.

In that bugzilla, it was mailContextMenus.js (as opposed to
nsContextMenus.js).

Sorry about the confusion.

One more bugzilla to file from the same test that times out.:
I have begun to see the following error since I refreshed
the source at the end of September:
JavaScript strict warning: resource://gre/modules/FormHistory.jsm, line
639: ReferenceError: reference to undefined property change.guid

TIA

On 2014/10/13 11:45, Boris Zbarsky wrote:
> On 10/12/14, 9:54 PM, ISHIKAWA, Chiaki wrote:
>> In this bugzilla, line 693 is actually out of range for the file reported.
>> The file is much shorter.
> 
> Is it?  The error is in nsContextMenu.js but your comments in the bug
> are about mailContextMenus.js, which is not at all the same thing.
> 
>> Worse, in another part of the log, I see line 694 reported for this
>> same error.
> 
> In the same file?
> 
> What I recommend doing is opening the exact chrome:// URI involved in
> view-source and going to the relevant line.  That will tell you what the
> file actually looks like in your install.  This matters, because during
> the build these files are sometimes preprocessed, have comments
> stripped, etc, which will make the line numbers the JS engine sees not
> match what's in the source tree (but match what's actually available if
> you click the error link).
> 
>> chrome://messenger/content/nsContextMenu.js, line 103: ReferenceError:
>> reference to undefined property this.onEditableArea
>>
>> When I look at line 103, the line again is an empty line
> 
> http://mxr.mozilla.org/comm-central/source/mail/base/content/nsContextMenu.js#103
> sure isn't.
> 
> Are you looking at the right file?
> 
>> I suspect that the line numbers are incorrect in these cases.
> 
> I doubt it.
> 
> -Boris
> 

_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to