Seems like this wouldn't accurately reflect agro from live as there is no additional agro for crits, however with the code structured like that, on crits your hate would be doubled. Also Im pretty sure that agro on live is based on weapon dmg, not actual dmg, which is why war weapons from recent expansions have less dmg/dly with relatively high elemental dmg tacked on(
The Keen Edge of Discord vs
Skulltutor). This keeps the war agro high but their dps in check, however that is not entirely relevant to your change and seems to be in the code already.
That said, it should be a relatively easy effect to test(feel doesnt really cut it, especially with 15%), use an item like
Raex's Chestplate of Destruction and check the hatelist entries with and without it.