Charts - remove all lint errors

Fixes #2533120 
Only issue is final commit. 
Using strict equality in doing string comparisons for user input data. If there are any use cases in the wild using new String("foo"), they will need to change. I'm ok with it. If someone is not speak up. Strict equality is the preferred method of comparing strings and the likelihood of someone using new String instead of string literal is slim. 

Feel free to pipe up if you disagree.