Clippy fixes

[aknarts]

This is mostly consideration but this passes following clippy checks

-W clippy::all -W clippy::pedantic -W clippy::nursery -W clippy::cargo -A clippy::cargo-common-metadata -A clippy::module-name-repetitions -A clippy::must-use-candidate -A clippy::let-underscore-drop -A clippy::needless-pass-by-value -A clippy::cast-sign-loss -A clippy::cast-possible-wrap -A clippy::use-self -A clippy::implicit-hasher

Notable allows:

• cast-sign-loss/cast-possible-wrap - disabled because we are getting values from JS, and this is for sizes, I do not think that window size will overflow i32 anytime soon
• implicit-hasher - while it would be nice to allow other hashers this overcomplicates the fixes
• use-self - happens in web binds mostly and there is no way to fix this

This probably needs to run tests and make sure it does not break anything, i.e. feel free to reject this(plus it is big)

[jetli]

Thanks!