chore: deduplicate FillTxEnv traits

[klkvr]

Right now we have 2 identical FillTxEnv traits with different implementations for TransactionSigned

This PR only keeps the one defined on reth_primitives_traits and updates impl for it to be the same as the one that was used before

[codspeed-hq]

CodSpeed Performance Report

Merging #13551 will degrade performances by 20.23%

_{Comparing klkvr/dedup-fill-traits (32b2633) with main (c6e38be)}

Summary

⚡ 1 improvements
❌ 1 regressions
✅ 75 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

	Benchmark	main	klkvr/dedup-fill-traits	Change
⚡	`hash builder[init size 10000	update size 100	num updates 1]`	10.1 ms
❌	`sparse trie[init size 10000	update size 100	num updates 3]`	10.1 ms

[mattsse]

cool,

this trait itself is a bit horrible and only really necessary because we usually call this via tx.fill(evm.tx_env_mut()) which should change shortly