Unified Diff: lib/contentPolicy.js
Issue 29329399:
Issue 3208 - Remove the bug 467514 work-around (link clicks in frames) (Closed)
Patch Set:
Created Oct. 26, 2015, 8:15 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/contentPolicy.js |
| =================================================================== |
| --- a/lib/contentPolicy.js |
| +++ b/lib/contentPolicy.js |
| @@ -546,23 +546,16 @@ var PolicyImplementation = |
| } catch(e2) {} |
| if (!newLocation) |
| return; |
| let wnd = Utils.getRequestWindow(newChannel); |
| if (!wnd) |
| return; |
| - if (contentType == Policy.type.SUBDOCUMENT && wnd.parent == wnd.top && wnd.opener) |
| - { |
| - // This is a window opened in a new tab miscategorized as frame load, |
| - // see bug 467514. Get the frame as context to be at least consistent. |
| - wnd = wnd.opener; |
| - } |
| - |
| if (contentType == Policy.type.POPUP && wnd.opener) |
| { |
| // Popups are initiated by their opener, not their own window. |
| wnd = wnd.opener; |
| } |
| if (!Policy.processNode(wnd, wnd.document, contentType, newLocation, false)) |
| result = Cr.NS_BINDING_ABORTED; |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
