OS/2 - convert wdgtos2.dll into a XPCOM module

VERIFIED FIXED

Status

()

VERIFIED FIXED
18 years ago
18 years ago

People

(Reporter: achimha, Assigned: achimha)

Tracking

Trunk
x86
OS/2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(5 attachments)

(Assignee)

Description

18 years ago
I started work on widget/src/os2 to convert it into a real XPCOM module like
the Xlib version. I think I got it almost right but not fully. I need some
help I guess. It builds fine but when starting Mozilla, I get
nsNativeComponentLoader: GetFactory(WDGTOS2) Load FAILED with error:
Range error
The classes it exports can be found in components.reg (searched for
contract ids) so that seems to have worked fine.
(Assignee)

Comment 1

18 years ago
Created attachment 27410 [details] [diff] [review]
the easy part, mark as a component
(Assignee)

Comment 2

18 years ago
Created attachment 27411 [details] [diff] [review]
whee - the diff is larger than the file...
(Assignee)

Comment 3

18 years ago
Yay, I got it working! I had to remove the old style registration in the
bootstrap code for the widget library on OS/2. Isn't it strange that we include
code in /mozilla/webshell/test/viewer in the retail build? I would have never
found this without LXR...
I'll attach a new diff for the widget/src/os2 code and for the bootstrap
code. Makefile.in patch is still valid.
CC'ing blizzard and pavlov cause they helped with bug 68343 which was the
same thing for gfx_os2.
(Assignee)

Comment 4

18 years ago
Created attachment 27413 [details] [diff] [review]
new diff with missing components added and nsTimer contract id corrected
(Assignee)

Comment 5

18 years ago
Created attachment 27414 [details] [diff] [review]
handle OS/2 like Unix for widget library - we're a full blown XPCOM component now!

Comment 6

18 years ago
I'm seeing two problems with this.

1. Sidebar doesn't come up at first.

2. Windows are positioned with titlebar off the screen.

Are you seeing this Achim?
(Assignee)

Comment 7

18 years ago
Mike, I'm not seeing any of your problems. Technically, the only thing this
change could affect is instantiating some contained classes. This would be
visible in the debug build console log (I first had incorrect timer contract
ID because timer was the only thing that was instantiated using its contract
ID but I saw the error on the console).

Did you delete dist/bin/wdgtos2.dll and components.reg?
Assignee: mkaply → achimha
(Assignee)

Comment 8

18 years ago
Created attachment 27450 [details] [diff] [review]
modelled contract ids after gtk cause this looks better than xlib code

Comment 9

18 years ago
r=pavlov

Comment 10

18 years ago
Fix checked in
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 11

18 years ago
done
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.