Closed
Bug 93500
Opened 25 years ago
Closed 24 years ago
Coordinates for NPWindow messed up for Windowless plugins
Categories
(Core Graveyard :: Plug-ins, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.4
People
(Reporter: kevint, Assigned: peterlubczynski-bugs)
References
()
Details
(Keywords: topembed, Whiteboard: [a=chofmann])
Attachments
(1 file)
|
1.17 KB,
patch
|
Details | Diff | Splinter Review |
For windowless plugins, the 'y' coordinate of the NPWindow passed into
NPP_SetWindow() is incorrect; in my tests it was giving me something in the
million or billion range. Tracing it back to nsObjectFrame::DidReflow(), it
calls parentFrame->GetOffsetFromView(), which is where it seems to get the
wrong coordinate from.
|
Assignee | |
Comment 1•24 years ago
|
||
*** This bug has been marked as a duplicate of 93056 ***
Status: UNCONFIRMED → RESOLVED
Closed: 24 years ago
Resolution: --- → DUPLICATE
|
|
Reporter | |
Comment 2•24 years ago
|
||
I discovered that this is actually not a duplicate of 93506, but the problem
boils down to the fact that the 'x' and 'y' parameters of NPWindow in npapi.h
were changed from int32 to uint32 - so if the y coordinate of a windowless
embed is negative, it'll come in as 0xFFFFFFFE, for example. I think using
int32 here is probably appropriate.
Status: RESOLVED → UNCONFIRMED
Resolution: DUPLICATE → ---
|
|
Assignee | |
Comment 3•24 years ago
|
||
Andrei, what do you think about the change?
Status: UNCONFIRMED → NEW
Ever confirmed: true
|
|
Assignee | |
Comment 5•24 years ago
|
||
|
|
Assignee | |
Comment 6•24 years ago
|
||
Reassign to myself. Seeking reviews.
|
|
||
Comment 8•24 years ago
|
||
sr=attinasi
|
|
Assignee | |
Comment 9•24 years ago
|
||
Patch in trunk. Seeking '+' for branch.
Status: NEW → ASSIGNED
Keywords: review
Whiteboard: [seeking reviews] → [patch in trunk][seeking approval]
|
||
Comment 10•24 years ago
|
||
a=chofmann for the 0.9.2 branch. thanks
|
|
Assignee | |
Updated•24 years ago
|
Whiteboard: [patch in trunk][seeking approval] → [patch in trunk][a=chofmann]
|
|
Assignee | |
Comment 11•24 years ago
|
||
Patch in branch, marking FIXED.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
|
|
||
Comment 12•24 years ago
|
||
verif that patch went in trunk as version 3.13
Whiteboard: [patch in trunk][a=chofmann] → [verif in trunk][a=chofmann]
|
|
||
Comment 13•24 years ago
|
||
verif patch in trnk. marking as such
Status: RESOLVED → VERIFIED
Whiteboard: [verif in trunk][a=chofmann] → [a=chofmann]
|
|
||
Comment 14•24 years ago
|
||
meant branch...
|
||
Updated•4 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•