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)
Firefox
Menus
Tracking
()
RESOLVED
FIXED
Firefox 39
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
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8566604 -
Flags: review?(mconley)
Updated•9 years ago
|
Assignee | ||
Comment 2•9 years ago
|
||
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.
Assignee | ||
Comment 3•9 years ago
|
||
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-
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Updated•9 years ago
|
Attachment #8566604 -
Attachment is obsolete: true
Assignee | ||
Updated•9 years ago
|
Attachment #8566902 -
Attachment is obsolete: true
Assignee | ||
Comment 4•9 years ago
|
||
Fixed in Bug 1133577.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•9 years ago
|
status-firefox39:
--- → fixed
Target Milestone: --- → Firefox 39
You need to log in
before you can comment on or make changes to this bug.
Description
•