Unified Diff: safari/include.youtube.js
Issue 29353237:
Issue 4361 - Pass all parameters to initEvent (Closed)
Patch Set:
Created Sept. 15, 2016, 5:57 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« safari/ext/content.js ('K')
|
« safari/ext/content.js ('k')
|
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: safari/include.youtube.js |
| diff --git a/safari/include.youtube.js b/safari/include.youtube.js |
| index 33011d13f3050e5290b843e4f68c5ff9deeefd24..1a8c8ee19961b3edc0c6555a997cbf55d497b9ce 100644 |
| --- a/safari/include.youtube.js |
| +++ b/safari/include.youtube.js |
| @@ -23,7 +23,7 @@ |
| try |
| { |
| var beforeLoadEvent = document.createEvent("Event"); |
| - beforeLoadEvent.initEvent("beforeload"); |
| + beforeLoadEvent.initEvent("beforeload", false, true); |
|
Sebastian Noack
2016/09/16 12:42:52
Any reason, we don't simply use ext.backgroundPage
kzar
2016/09/16 13:00:43
I'm not sure, I can't remember to be honest.
Sebastian Noack
2016/09/16 13:29:03
Then any reason you don't change it? That way this
kzar
2016/09/16 14:04:24
Done.
|
| if (safari.self.tab.canLoad(beforeLoadEvent, |
| {category: "request", |
| payload: {type: "prefs.get", |
« safari/ext/content.js ('K')
|
« safari/ext/content.js ('k')
|
no next file »
|
no next file with comments »
