Patch 2.2 re-worked some legendary items, among them the Broken Promises ring. It's new affix states:
After 5 consecutive non-critical hits, your chance to critically hit is increased to 100% for 3 seconds.
So, what counts as a hit? I know regular primary attacks will. But do gem procs, such as Miranae's smite? The Wreath of Lightning's AoE zapping? Pet attacks? Do AoE attacks count only once, or once per monster hit? What about sustained skills, such as Sweeping Wind?
Way of the Hundred Fists also has an attack that does multiple hits. You can easily see it calculate crits separately per hit, so do those count as separate hits for the purposes of this affix?
I can see a non-crit build, buffed with as much attack speed as you can stack, keeping this proc up almost 100% of the time.
Best Answer
I did some testing earlier today and found there does seem to be a bit of an internal cooldown. After 3-4 seconds of your crit chance buff going down, you can repeat the process of activating the ring.
I play monk and tried with a few spirit generators and none of them seemed to be a clear cut best choice, although I do need to keep testing. To answer your question about hundred fists (and each spirit generator), they are all combos with 3 hits. So even though it shows a lot of little punches, it only counts as 3 hits per sequence. Hundred fists is cool though, because your ring buff activates right before the 3rd part of your 2nd sequence which is a flurry of little punches. As long as you didn't crit in the middle of it, that is.
Bonus: I did a little more research and it seems that only skills performed by you will count towards the ring proc, so pets/item specials probably won't affect it. But sweeping wind does proc it...and it can't crit ;)