Re: [9front] adding javascript enable to netsurf

[kokamoto@hera.eonet.ne.jp]

[-- Attachment #1: Type: text/plain, Size: 74 bytes --]

Thanks a lot!

Yes, it seems to hit my case.
I'll see it tommorow.

Kenji

[-- Attachment #2: Type: message/rfc822, Size: 3584 bytes --]

From: jamos@oboj.net
To: 9front@9front.org
Cc: kokamoto@hera.eonet.ne.jp
Subject: Re: [9front] adding javascript enable to netsurf
Date: Wed, 11 Mar 2020 10:29:03 +0100
Message-ID: <80e9edc42326584c4073d484bef78e14@oboj.net>

Reverse engineering a Linux executable, there is indeed a
javascript_init() defined. It looks like it comes form the
following object file:

nm netsurf/build/Linux-gtk/content_handlers_javascript_content.o | grep 
javascript_init
0000000000000150 T javascript_init

I cannot either find the proper declaration for the function
in the source, but maybe it is autogenerated somehow. In
'netsurf/content/content_factory.h' I find the macro:


#define CONTENT_FACTORY_REGISTER_TYPES(HNAME, HTYPELIST, HHANDLER)      
\
nserror HNAME##_init(void) { … }

That is used in 'netsurf/content/handlers/javascript/content.c':

CONTENT_FACTORY_REGISTER_TYPES(javascript, javascript_types, 
javascript_content_handler);

I don’t have the time right now to investigate further, but maybe
those findings can lead you a bit further. Duktape can be
enabled/disabled both in compile time and runtime too, I think.

Jonas

On 2020-03-11 07:36, kokamoto@hera.eonet.ne.jp wrote:

> I'm still struggling this.
> Error message says:
> javascript_init: redifinition: javascript_init
> (4481) TEXT javascript_init t0(SB). R0, $16
> 
> javascript_init is declared only once in
> ./netsurf/content/handlers/javascript/content.h:
> nserror javascript_init(void);
> 
> However, I cannot find the body of that function anywhere...
> 
> The javascript_init is called from 
> ./netsurf/content/handlers/javascript/duktape/dukky.c like:
> 
> /* exported interface documented in js.h */
> void js_initialise(void)
> {
> /** TODO: Forces JS on for our testing, needs changing before a release
> * lest we incur the wrath of others.
> */
> /* Disabled force-on for forthcoming release */
> /* nsoption_set_bool(enable_javascript, true);
> */
> javascript_init();  <====
> }
> 
> Please let me know the meaning of that comment, I'm not native English 
> user,
> then, it is difficult to understand why then javascript_init() is used 
> here...
> 
> Kenji