Closed Bug 1134708 Opened 9 years ago Closed 9 years ago

[e10s] "This Frame > Show/Open" commands causes unsafe CPOW usage warning

Categories

(Firefox :: Menus, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 39
Tracking Status
e10s m8+ ---
firefox39 --- fixed

People

(Reporter: Kwan, Assigned: Kwan)

References

Details

Attachments

(2 obsolete files)

+++ This bug was initially created as a clone of Bug #1133577 +++

STR:

1) Visit a site with frames on it in an e10s window
2) Right-click on a frame, and choose "This Frame" then any of "Show Only[...]", "Open Frame in New Tab", "Open Frame in New Window"

This causes some "unsafe CPOW usage" warnings in the Browser Console.

In browser/base/content/nsContextMenu.js:

[all from uses of doc.location.href, doc.referrer, or doc.characterSet]

patch incoming
Attachment #8566604 - Flags: review?(mconley)
Inconveniently, I made this patch on a slightly old m-c, and after the backout of bug 1075670 it no longer applies cleanly.
Here's one that does.
Blocks: 1133577
Comment on attachment 8566604 [details] [diff] [review]
Make context-menu open-frame commands e10s safe

Clearing review because I need to make a patch with saner naming.  Having frameURL separate from doc.location.href (which is needed for the regular open link) doesn't seem to make sense unless there's a time when we need both.  And thus the frame check if() is probably unnecessary.
Might swap the ordering of the patches as well, or just combine them.
Attachment #8566604 - Flags: review?(mconley) → review-
No longer blocks: 1133577
Depends on: 1133577
Attachment #8566604 - Attachment is obsolete: true
Attachment #8566902 - Attachment is obsolete: true
Fixed in Bug 1133577.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 39
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: