Replace common.NewHash and common.Hash() with common.BytesToHash

[kishansagathiya]

Issue summary

• common.BytesToHash, common.NewHash and common.Hash changes bytes to hash (does not hash, just converts). Both of these functions are used interchangeably, but common.BytesToHash puts 0 bytes in the front and common.NewHash puts 0 bytes in the back.

• Say []byte argument is 0x44102080304545b0b6579496e736964656368696c6414766f696c61
common.BytesToHash will give hash like 0x00000000044102080304545b0b6579496e736964656368696c6414766f696c61 and common.NewHash will give a hash like 0x44102080304545b0b6579496e736964656368696c6414766f696c61000000000

• We should pick one of them and use it across the project to avoid any mismatching in the value of hash. I have gone with common.BytesToHash so far.

Other information and links

• inspired from this pr fix(lib/trie): Make sure writing and reading a trie to disk gives the same trie and cover more store/load child trie related test cases #2302