search

kustodian / extended-statusbar

Firefox addon which adds a statusbar with speed, percentage, time and loaded size (similar to Opera's one)
https://addons.mozilla.org/en-US/firefox/addon/extended-statusbar/
GNU General Public License v3.0
18 stars 6 forks source link

Incompatible with e10s - "aBrowser.contentDocument is null" #45

Closed Dook1 closed 7 years ago

Dook1 commented 8 years ago

Incompatible with e10s - "aBrowser.contentDocument is null"

TriMoon commented 7 years ago

Why don't you just fix The violating line to check for existence? See: my source-comment

adoxa commented 7 years ago

Done (using yet another method :)).

TriMoon commented 7 years ago

@adoxa Done (using yet another method :)).

Unfortunately IMHO https://github.com/kustodian/extended-statusbar/commit/bdf84e202b645f64246fb67412462dd4023eb397 won't fix it properly because it will introduce another bug... Reading the code that follows shows it will check for not null, undefined == ! Null

adoxa commented 7 years ago

According to the console, undefined does equal null.