Returning const references for accessors

Lots of cleaning up in MOOSE internals
closes #3357 #3358
You're not catching the return value as a reference here... so this is doing a copy...