Closed Bug 59521 Opened 24 years ago Closed 24 years ago

JavaDOM doesn't work with RTM branch

Categories

(Core Graveyard :: Java APIs to WebShell, defect, P3)

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: edburns, Assigned: edburns)

Details

(Whiteboard: suntrak-n6)

Attachments

(3 files)

JavaDOM doesn't work with RTM branch
Accept. 

I un-commented the line that disables DOMViewer in EMWindow, and found that DOM 
Viewer works on Win32.  Can someone please try it on Unix?
Status: NEW → ASSIGNED
tried this on solaris. works, but a text field incorrectly drawn. didn't see
this before the switch to jdk1.3.0_01. not sure if jdk is the cause though.

Could Geetha or Raju please confirm this.
Attached patch cvs diff -uSplinter Review
fix for getting JavaDOM working through Webclient on RTM branch is attached. Fix
is required for Solaris (JavaDOM works on WinNT with or without the fix)
Ashu, can you please move the JNI stuff inside the handleEvent() method?  That
way, CurrentPageActionEvents.h won't have to include jni.h.  I want to limit the
number of files that get included in .h files.  Also, do your best to isolate
all JNI calls inside #ifdef BAL_INTERFACE #else #endif blocks to ease the
transition to StarOffice.  Please post a second iteration and I'll review it.

Thanks,

Ed
Posted second iteration - cleaner fix
Created a new util_CallStaticObjectMethodlongArg function in jni_util and
included jni_util in CurrentPageActionEvents.h (this cannot be avoided). The
handleEvent for GetDOMEvent uses this util_ function. The util function is
placed in #ifndef BAL_INTERFACE blocks for easier transition to StarOffice
Ed, I need ra=edburns for the second iteration of fix   ---Ashu
What about CurrentPageImpl.cpp?
r=edburns
Adding Status white board keyword to track all existing Webclient bugs
Whiteboard: suntrak-n6
Ed, pls close this Bug. I have already checked in the fixes with ra=edburns
fixed
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Obsolete bug, should be closed. 
Verified.
Status: RESOLVED → VERIFIED
QA Contact: geetha.vaidyanaathan → avm
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: