Unified Diff: lib/popupBlocker.js
Issue 29417555:
Issue 5130 - Changes url protocol permissions to all urls (Closed)
Patch Set: change to explicit urls at handler registration
Created April 19, 2017, 12:04 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« lib/devtools.js ('K')
|
« lib/devtools.js ('k')
|
metadata.chrome » ('j')
|
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/popupBlocker.js |
| =================================================================== |
| --- a/lib/popupBlocker.js |
| +++ b/lib/popupBlocker.js |
| @@ -95,17 +95,17 @@ |
| chrome.webNavigation.onCreatedNavigationTarget.addListener(details => |
| { |
| if (!hasLoadingPopups()) |
| { |
| chrome.webRequest.onBeforeRequest.addListener( |
| onPopupURLChanged, |
| { |
| - urls: ["<all_urls>"], |
| + urls: ["http://*/*", "https://*/*", "ws://*/*", "wss://*/*"], |
|
Sebastian Noack
2017/04/19 12:10:11
Same here.
Jon Sonesen
2017/04/19 12:22:27
Done.
|
| types: ["main_frame"] |
| } |
| ); |
| chrome.webNavigation.onCommitted.addListener(onPopupURLChanged); |
| chrome.webNavigation.onCompleted.addListener(onCompleted); |
| chrome.tabs.onRemoved.addListener(forgetPopup); |
| } |
« lib/devtools.js ('K')
|
« lib/devtools.js ('k')
|
metadata.chrome » ('j')
|
no next file with comments »
