Passthrough fix

[jbylsma]

I was having passthrough issues like issues #118, #119, and #121. I bisected commits backed and narrowed the problem down to 28fe4af, where keyup's pass assignment was converted to a function. I found two issues:

• The final passthrough test in the pass assignment previously had both the digit test and the mapping test connected: both had to be false in order to return true. The updated code had disconnected them, which I'm guessing was because of the trickiness of the parentheses. I tried to correct this following the established code styles.
• The get method in mapHive would return null is skipPassthrough was true and the mapping's passthrough property was false. I'm pretty sure the map.passthrough should have been true instead.

Although I haven't done any Firefox plugin development before and I didn't quite following the abstraction changes in Pentadactyl, I'm fairly confident that the logic changes I've made are correct. I've A/Bed against previous successful commits and util.dumped a lot of vars to make sure things seem right.

[lichray]

Confirmed to work with github and google search.

[hrzhu]

Tried your branch, it fixed the pass-through issue on github.

[PTNobel]

Fixed GitHub, Google, and Firefox's built-in PDF viewer (I think it's named pdf.js) for me.

[woutershep]

Can confirm that it fixes #121 for me. Haven't noticed any negative side-effects at least. I have switched from a local version where i had 28fe4af reverted, to this pr merged into master, and couldn't tell any difference.

[deewakar]

Fixes Reddit ( Reddit Enhancement Suite) too.

[abazishkumbin]

Are there any plans to merge this?

[arxanas]

+1, also suffering from this.

[jbylsma]

The commit 4904fb5bff72e0d45fea353372ccf9c3061722f8, which was recently applied to master, applies the same logic as this pull request, so I'll go ahead and close this. Thank you everyone for testing!