migration from EDProducer to stream::EDProducer (BeamSpotOnlineProducer)

as reported by @fwyzard
https://docs.google.com/spreadsheets/d/1D9khDvOl05WEbznsUPCusEbrHeep9hh8cXedMXX09Gc/edit#gid=0
there are modules used @HLT which still need to be migrated and be multithread safe

this is the migration of the module BeamSpotOnlineProducer
I take that back, it is just used to make the code verbose. Untracked is the proper type.