Closed
Bug 36698
Opened 25 years ago
Closed 24 years ago
PSM displays wrong for frames named "content" or "left"
Categories
(Core :: Layout: Images, Video, and HTML Frames, defect, P3)
Core
Layout: Images, Video, and HTML Frames
Tracking
()
VERIFIED
WORKSFORME
Future
People
(Reporter: junruh, Assigned: danm.moz)
References
()
Details
(Whiteboard: [nsbeta2-][rtm-])
1) Visit https://enigma.barclaycard.co.uk. Wait for full download.
2) Click on the lock icon.
What is expected: PSM would appear with relevant info.
What happens: PSM appears, but the relevant info is in a frame of the Barclay's
website.
Build 2000042009. WinNT and Linux.
Comment 1•25 years ago
|
||
does this happen on any https url?
Reporter | ||
Comment 2•25 years ago
|
||
I've only seen it on this particular URL
Comment 3•25 years ago
|
||
nsbeta2- unless the problem turns out to be more widespread
Whiteboard: [nsbeta2-]
Reporter | ||
Comment 4•25 years ago
|
||
Reassigning to javi during dougt's absence for the next three weeks.
Assignee: dougt → javi
Comment 5•25 years ago
|
||
This is actually a bug in the DOM code which tends to get confused in certain
cases where frames are used. It'll fill in the wrong frame with content from a
different window. Please re-assign.
Reporter | ||
Comment 6•25 years ago
|
||
Reassigning
Assignee: javi → alecf
Component: Security: Crypto → DOM Viewer
Comment 7•25 years ago
|
||
why is this assigned to me? this is a browser bug. reassigning to don
Assignee: alecf → don
If this is a Frames or DOM problem, then why do I have it?
Over to Eric Pollmann for triaging ... 'cause I have no clue who should get
this.
Component: DOM Viewer → HTMLFrames
Target Milestone: M18 → ---
Comment 10•25 years ago
|
||
OK, let's REALLY re-assign it this time.
Assignee: don → pollmann
QA Contact: junruh → petersen
Comment 11•25 years ago
|
||
Reduced test case (does not require https):
-------- http://blueviper/frames/contentframe.html
<frameset cols="*,*">
<frame name=content src="about:blank">
<frame name=foo src="about:blank">
</frameset>
--------
The problem is that if there is a frame named "content" in the document, when
PSM tries to update it's frame named "content", it updates the document's frame
instead of it's own.
CC'ing danm - guru of frame name resolution :)
Comment 12•25 years ago
|
||
This means that if any page on the web has a frame named "content" or a frame
named "left" (or heaven forbid both!), chunks of PSM will display in the
brwoser frame by that name and not in the dialog as they should. I think that
this could actually be quite common... Re-nominating for beta2 with this new
information.
(There might also be a problem if the frame is named "top" but I've not been
able to verify one yet)
Status: NEW → ASSIGNED
Summary: Weird PSM display behavior → PSM displays wrong for frames named "content" or "left"
Whiteboard: [nsbeta2-]
Assignee | ||
Comment 13•25 years ago
|
||
The frame-named-content-collision problem was fixed, albeit in a rather cheesy
way, on June 21st (see bug 33650). Does that kill this bug, too?
Comment 14•25 years ago
|
||
This bug is present with today's build.
I'm pretty sure this is a separate problem - PSM has a frame named "content" in
it, and a frame named "left". If you click on the Lock icon, and the page you
are viewing in the browser has either a frame named "content" or a frame named
"left", the frames in the page you are viewing in the browser will be updated,
instead of the frames in the PSM dialog. It seems like the search for the named
frame is not starting in the right window or something like that???
Comment 15•25 years ago
|
||
That said, PSM could also use the cheese fix and rename these frame to
_psm_content, _psm_left, and _psm_top. ;)
Comment 16•25 years ago
|
||
Handing off to Dan. Per our conversation - it seems like the problem may be a
failure to search *our* window for the named frame before defering to the
windows mediator and iterating through all windows looking for the named frame.
Let me know if I can help!
Assignee: pollmann → danm
Status: ASSIGNED → NEW
Updated•25 years ago
|
Whiteboard: [nsbeta2-]
Comment 17•25 years ago
|
||
Putting on [nsbeta2-] radar. Not critical to beta2.
Assignee | ||
Comment 18•25 years ago
|
||
This bug seems like a symptom of our incorrect search order when looking for
windows to open. It'll cause edge-case problems with certain rare bits of JS,
like in our PSM window. Hmmmm. Borderline. But marking "future" because of
impending release time constraints.
Target Milestone: --- → Future
Reporter | ||
Comment 19•25 years ago
|
||
This now only happens when opening the Security manager from the Tasks menu.
Keywords: rtm
Reporter | ||
Comment 20•25 years ago
|
||
*** Bug 57334 has been marked as a duplicate of this bug. ***
Comment 21•25 years ago
|
||
Hmm, this got nominated for rtm at the last minute, but the last comment implies
a workaround, so rtm-
Whiteboard: [nsbeta2-] → [nsbeta2-][rtm-]
Reporter | ||
Comment 22•25 years ago
|
||
*** Bug 63255 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 23•25 years ago
|
||
*** Bug 63255 has been marked as a duplicate of this bug. ***
Comment 24•24 years ago
|
||
This also happens when there is an open pop-up window.
For example, goto http://www.cnn.com and vote in the "Quick vote" to have a pop
up window.
If the pop-up window remains open, PSM displays actual info in that pop-up
window, and the menu tabs in its own pop-up window.
Is this an independent bug?
Comment 25•24 years ago
|
||
The cnn popuppoll window contains three frames named "top", "content" and
"back". It is undoubtedly the window named "content" that is causing the
problem with PSM, meaning that that is exactly the same as this problem, not a
separate issue.
FWIW, when I had the popup window up, and clicked on the lock in *another*
window, nothing bad happened. ???
Comment 27•24 years ago
|
||
I think this bug was fixed when PSM was re-written as I no longer the problem.
Reporter | ||
Comment 28•24 years ago
|
||
Marking worksforme. PSM is totally different now, plus the page no longer
exists.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → WORKSFORME
Updated•7 years ago
|
Product: Core → Core Graveyard
Updated•7 years ago
|
Component: Layout: HTML Frames → Layout: Images
Product: Core Graveyard → Core
You need to log in
before you can comment on or make changes to this bug.
Description
•