common: Cleanup backtrace.
common: Cleanup backtrace.

There was a bool field that was caching results of 'is_backtrace_needed'
lookups.   Now that field is gone, and 'is_backtrace_needed' is called
repeatedly.  I guess it's not that bad (it boils down to a lookup in an
std::set), but that function was also printing notices for functions
with enabled stack trace recording, and now those messages are printed
repeatedly as well.  The following patch removes the notices completely
and slightly cleans up the surrounding code.

surround-io/apitrace,EoD/apitrace,surround-io/apitrace,tuanthng/apitrace,apitrace/apitrace,tuanthng/apitrace,EoD/apitrace,tuanthng/apitrace,EoD/apitrace,swq0553/apitrace,surround-io/apitrace,trtt/apitrace,joshua5201/apitrace,PeterLValve/apitrace,apitrace/apitrace,EoD/apitrace,EoD/apitrace,trtt/apitrace,swq0553/apitrace,surround-io/apitrace,schulmar/apitrace,joshua5201/apitrace,swq0553/apitrace,PeterLValve/apitrace,schulmar/apitrace,joshua5201/apitrace,swq0553/apitrace,schulmar/apitrace,apitrace/apitrace,joshua5201/apitrace,tuanthng/apitrace,apitrace/apitrace,joshua5201/apitrace,trtt/apitrace,tuanthng/apitrace,swq0553/apitrace,schulmar/apitrace,PeterLValve/apitrace,schulmar/apitrace,trtt/apitrace,PeterLValve/apitrace,surround-io/apitrace,trtt/apitrace