Closed Bug 7796 Opened 26 years ago Closed 26 years ago

[PP]nsDeviceContextMac::GetDeviceSurfaceDimensions is not impliment on the mac

Categories

(Core :: Layout, defect, P3)

PowerPC
Mac System 8.5
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: davidm, Assigned: beard)

References

Details

The following diff makes the mac work in a limited fashion
	Index: nsDeviceContextMac.cpp
===================================================================
RCS file: /cvsroot/mozilla/gfx/src/mac/nsDeviceContextMac.cpp,v
retrieving revision 1.51
diff -r1.51 nsDeviceContextMac.cpp
324,325c324
<   aWidth = 1;
<   aHeight = 1;
---
> 	GDHandle device = ::GetMainDevice();;
326a326,327
> 	aWidth = (**device).gdRect.right;
> 	aHeight = (**device).gdRect.bottom;

but someone really should do the work to be multimonitor friendly. As there is no
gfx component I am assiging to layout
Blocks: 1877
Note that the above code is wrong since from some reason the PRInt32 is really
supposed to be one of the fixed point values. I don't know why.
Assignee: rickg → beard
assigning to beard per rickg
Target Milestone: M8
targetting m8 per rickg
Status: NEW → ASSIGNED
Summary: nsDeviceContextMac::GetDeviceSurfaceDimensions is not impliment on the mac → [PP]nsDeviceContextMac::GetDeviceSurfaceDimensions is not impliment on the mac
This issue is occuring in the June 30th (1999063009)
moving to m9. beard's on vacation
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
I put in a quick fix that just uses the GrayRgn (the region encompassing the
entire desktop) and returns the dimensions of its bounding box. Checked fix in as
mozilla/gfx/src/mac/nsDeviceContextMac.cpp,1.52.
Status: RESOLVED → VERIFIED
Based on Bread's comments, marking as verified fixed.
You need to log in before you can comment on or make changes to this bug.