Saturday, 7 July 2012

Re: JavaScript

On Sat, 7 Jul 2012 18:17:12 +0200, m0n0 wrote:

> I have no problems with version 1.7 - no crashes or such.
> Are you using my spidermonkey-1.7 branch? It's not in sync with
> the master so you need pull in commits of the master.

I'm using master with the missing macros in my Spidermonkey jsapi.h.
I've just tried your branch, and although it doesn't crash it is
complaining with this error on my simple test:
(14.417069) javascript/jsapi.c js_reportError 59: <head>:1:ReferenceError: alert is not defined

This was working a while back with 1.50, however as you state, your
branch is out of sync, and libdom has had some changes (which I think
is what is causing my crash and is probably also responsible for
alert() now not working with the older code - although I haven't yet
tried linking back with 1.50)

If I sync to master I think it will start crashing again.

It's definitely something to do with Javascript. If I disable
Javascript in Choices it stops the crash. All my libraries are
up-to-date (I have double and triple checked this and rebuilt them
all). I think the stack trace might be a red herring as it is
following through parse_chunk, which I don't think has changed and I
find it very unlikely this has suddenly decided to attempt to jump to
a NULL address. It's possible that memory is getting corrupted which
is then causing NetSurf to fall over.

My suspicion is libdom purely because everything was working before
the libdom binding changes, and that's the only library which has
changed significantly.

Chris

No comments:

Post a Comment