Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[Firefox addon] Replace a bind(this) statement with an arrow function in PdfStreamConverter.jsm (issue 8343)

Browse Source 
Please note that I used the addon debugger to set a breakpoint in the `unload` function, in order to ensure that `this` still correctly refers to the `FindEventManager` scope.
This commit is contained in:
Jonas Jenwald 2017-04-30 13:21:37 +02:00
parent 00d67371ec
commit 8e976d7110
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
extensions/firefox/content/PdfStreamConverter.jsm
View File

@ -804,10 +804,10 @@ class FindEventManager {
} }
bind() { bind() {
var unload = function(e) { var unload = (evt) => {
this.unbind(); this.unbind();
this.contentWindow.removeEventListener(e.type, unload); this.contentWindow.removeEventListener(evt.type, unload);
}.bind(this); };
this.contentWindow.addEventListener("unload", unload); this.contentWindow.addEventListener("unload", unload);
// We cannot directly attach listeners to for the find events // We cannot directly attach listeners to for the find events