105959 - port NPAPI tester plugin to other platforms

Status: RESOLVED DUPLICATE of bug 386676

(Core Graveyard :: Plug-ins, defect, P4)

Description

[av (gone)]

We nees a convenient tool which would allow to issue NPAPI calls and log the 
result for testing purposes. The place is mozilla/modules/plugin/tools/tester.

Comment 1

[Peter Lubczynski]

Tester plugin is there on windows and works pretty good.

Changing summary to port tester plugin to other platforms :)

Comment 2

[serge (gone)]

Attachment: patch to support NPN_ReqestRead() testing

I used it as a test case for bug 115891, bug 120411
av, peter would you please review this?

Comment 3

[av (gone)]

Comment on attachment 75699 [details] [diff] [review]
patch to support NPN_ReqestRead() testing

Wow! I even forgot I did not implement this. This is great. I guess the
following is not terribly necessary:

-DWORD convertStringToNPPVariable1(DWORD * pdw1);
+DWORD convertStringToNPPVariable1(DWORD * pdw1);;

And you can take a full advantage of the project not being part of the regular
build, just mention this fact in check in comments.

Comment 4

[av (gone)]

Comment on attachment 75699 [details] [diff] [review]
patch to support NPN_ReqestRead() testing

Sorry, forgot... r=av

Comment 5

[serge (gone)]

on the trunk w/o extra ; in tester/include/strconv.h
thanks.

Comment 6

[rubydoo123]

reassigning to anthony, Anthony you'll need to get the scoop from serge, av and
peterl

Comment 7

[Peter Lubczynski]

--->peterl

Comment 8

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

We have zero (none, bupkiss) plugin test coverage in our automated suite.  Poking this on the off chance that it still has the right assignee.

Comment 9

[Benjamin Smedberg]

I will offer to mentor a student willing to write/fix/extend the NPAPI tester plugin for the Google Summer of Code: http://wiki.mozilla.org/Community:SummerOfCode08:Brainstorming#Mozilla_Platform

I especially want comprehensive tests of the scriptable NPAPI.

Comment 10

[Johnny Stenback (:jst)]

Not going to block a minor release on this.

Comment 11

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

I didn't ask for it to block, I asked for it to be marked "wanted" as an indication that we'd take such a fix on the branch to give us some shred of test coverage there. :(

Comment 12

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

Wanted, but is anyone working on this? Josh?

Comment 13

[(not currently active) Ted Mielczarek]

Duping forward, we have a test plugin now. File new bugs for specific capabilities that would improve testing.