Closed
Bug 46253
Opened 25 years ago
Closed 24 years ago
Events for child windows of the security advisor are broken - can't edit certs; can't close windows
Categories
(Core Graveyard :: Security: UI, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: junruh, Assigned: ddrinan0264)
References
Details
(Whiteboard: [nsbeta3-][dogfood-][rtm need info])
1.) Visit http://junruh.mcom.com/sstests/hardobject.cacert to directly import a
new CA.
2.) At the Step 1 window, click on View.
3.) View the CA cert, then click OK to close the window.
What happens: The button does not close the window. Linux build 072209. WinNT
works OK.
Comment 1•24 years ago
|
||
Does the import action succeed? Also can you close the window by another means
(x box)? If the answer to these questions is yes, then I don't believe this is
a stop ship bug. Marking "need info".
Whiteboard: [NEED INFO]
Reporter | ||
Comment 2•24 years ago
|
||
The answer is yes to both, so not a stop ship bug.
Reporter | ||
Comment 5•24 years ago
|
||
The fact that the left mouse button does not work in the child window prevents
editing certs. Making 41926 a dupe of this bug.
Reporter | ||
Comment 8•24 years ago
|
||
Open the SA and open any child window like view the certificate. Now try to
close the view windows. You cannot close the window. You have to close the SA
first and then you can close the view window.
Adding rtm and dogfood keywords.
Comment 9•24 years ago
|
||
this happens on Mac too.
Severity: normal → critical
Hardware: PC → All
Summary: Linux-Cannot close View CA cert window with OK button. → Events for child windows of the security advisor are broken; can't close windows
Comment 10•24 years ago
|
||
This appears to be caused by us calling window->OpenDialog to load the security
advisor. As long as the security advisor is open, it's the only window that
processes mouse events. If I change the call to merely call window->Open, this
problem goes away, but the style sheets for the security advisor don't seem to
work as well anymore. (There shouldn't be underlines on any of the tabs after
clicing on another tab.)
Reporter | ||
Comment 11•24 years ago
|
||
This should be rtm++.
Comment 12•24 years ago
|
||
John - to confirm: with this bug, you cannot edit certs at all?
Thanks
Reporter | ||
Comment 13•24 years ago
|
||
I still cannot edit certs with the 101010 commercial branch Linux build.
Comment 14•24 years ago
|
||
yes with linux , you cannot any subsequent windows which comes up in the
Security Advisor. You cannot edit certs, do OCSP, restore gave me a problem
just now. why is this not [rtm need info]? I am nominating it and if anyone
feels it should not be ,please explain why.
Whiteboard: [nsbeta3-] → [nsbeta3-][rtm need info]
Comment 15•24 years ago
|
||
javi - will you get a chance to investigate this? We are getting close to rtm.
If you are not the right person, can you reassign this bug to the right person?
Summary: Events for child windows of the security advisor are broken; can't close windows → Events for child windows of the security advisor are broken - can't edit certs; can't close windows
Comment 16•24 years ago
|
||
This seems like very important rtm functionality, but folks don't typically have
to edit their certs very often (that I know of). This shouldn't stop day-to-day
usage of the product, which is what the Netscape dogfood rating is focused on.
I'm marking dogfood-minus for now.
If there are significant elements of testing that are blocked by this bug,
please re-nominate for dogfood
Whiteboard: [nsbeta3-][rtm need info] → [nsbeta3-][dogfood-][rtm need info]
Comment 17•24 years ago
|
||
this bug pretty much does not allow anything to be done. To name a few features
in Security Advisor that are affected are
viewing a certificate
change/Set Password
edit a cert
add external modules
change OCSP settings
backup and backup all
delete a certificate.
so this pretty much means that the security advisor cannot be used on linux by a
regular PSM user. I guess that means it is dogfood+ for a linux user.
Comment 18•24 years ago
|
||
regardless of the dogfood+/- status, this bug sounds like it needs to be fixed
for rtm to have a functional security advisor. Any updates to this, Javi?
Comment 19•24 years ago
|
||
This has got to be a dupe of another bug, cause I remember typing earlier today
that I didn't have the cycles to look into this.
I think the work ddrinan is doing for other bugs will fix this bug.
Re-assigning.
Assignee: javi → ddrinan
Comment 20•24 years ago
|
||
In my (successful) attempts to reproduce this bug, every case I've seen is one
where a nonmodal window is opened as an owned/transient/child window of a modal
window. I guess this isn't a well-known fact, because people try it all the time,
but you pretty much just can't do that. We can't handle it, and it's largely not
even our fault.
It's been a while since I've looked at the gtk source, and I can do that again
if you need me to, but I think on Linux builds the problem begins with gtk. I
believe gtk sets up an event filter to ignore windows other than the modal one.
Maybe we could hack around that -- we already are in one very specific case --
but I think the real solution is to just not get into that situation. (The Mac
has similar problems though in that case it's our (my) fault, not some underlying
toolkit's.)
I think we need to make the security window nonmodal, or stop opening nonmodal
browser windows on top of it. Is there any chance that can be the solution to
this bug?
Reporter | ||
Comment 21•24 years ago
|
||
*** Bug 56981 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 22•24 years ago
|
||
Javier checked in a fix for this last week for Linux. PSM UI on Linux is now
functional. I'm marking this fixed.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 23•24 years ago
|
||
Verified with the 10/19 commercial branch build.
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 24•24 years ago
|
||
Mass changing Security:Crypto to PSM
Component: Security: Crypto → Client Library
Product: Browser → PSM
Target Milestone: M18 → ---
Version: other → 2.1
Reporter | ||
Comment 25•24 years ago
|
||
Mass changing Security:Crypto to PSM
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•