Great results! :-) I had a quick look at the code; if I looked at it correctly,
there's some leftovers of the old method in normalize that should be cleaned up.
Perhaps we can also simplify the comment at the start of the method to remove
the mention about detecting new irrelevant things so that we don't forget it
later (after merging the issues for that, assuming we'll be able to do that)
when it's no longer relevant?
After the cleanup, I can have another look at the code if you want (for which it
would be best to start a pull request or upload it to Rietveld), but you can
also merge directly if you want.
|