Get rid of Savage-Blade. Useless as Warrior.
Haymaker on the other hand is not that bad. Flash is weak compared to Overpower, and it's also missing the blind effect, while the slow from Haymaker is another way to reduce incomming damage. I guess that one is personal preference.
Provoke is essential though. You have no chance in endgame content without it. Get it, keep it, love it.

When tanking you are the person who dicdates the speed of a run.
You decide how much you pull, where to tank and where to go. If others pull on purpose, let them tank the mobs.
When tanking you always have tank buff up and only you decide if you can drop it (Stone Vigil hard last boss for example).
When playing with other tanks, don't fight about aggro. If someone keeps trying to steal hate be smart and let them have it to prevent the mob from turning in circles (which drives Drg and Mnk really mad) and reduces incomming dmg for the party.
This also includes dodging AoE or cone attacks. always try to dodge in a way which prevents the mob from moving or turning and position it so melee can effectively use their skills.

You should keep aggro at any moment during a fight, except it's a tank swap.
If people continue to attack another mob than you and draw hate, set target marks.
If stupid people unload all their cooldowns right at pull and steal hate, let them have it.
Aggro management is a group effort.
You should do what you can to keep everything you're supposed to tank on you though. If you have problems with packs, Steel Cyclone offers great hate generation.
If you're really a good tank you know when you messed up.

Don't become an ego tank like so many others who can't bear not being MT and keep spamming hate combos to steal hate.
When you're OT and there is no need for you to tank anything, just disable defiance and, and that's the great thing about War compared to Pld, keep using the Storms debuffs on the mob.
They make the life of the MT easier and you don't generate more hate than any other DD.


Also, don't use any rotation guides!
As tank you often have to make quick decisions and can't just "finish the combo, then try to grab the mob which keeps smacking the half dead healer".
You will do alot of target swaps to keep an eye on enmity levels and use your tools to enable the rest of the party to do their jobs.

That's your job as tank. Not taking damage. Everyone can do that.
You're there to take as little damage as you can while keeping the enemies focus on you, so everyone else can do their thing.
That being said, it's a good thing if you play other classes too, because you will see for yourself what they require from the tank, to be effective.