PF cluster isolation for muons + TimeCleaned RecHit producer.

[HuguesBrun]

This PR will add the producers and filters to have the PF cluster isolation on muons at HLT.
The EgammaHLTEcal[Hcal]PFClusterIsolationProducer.cc are replaced by a template HLTEcal[Hcal]PFClusterIsolationProducer.cc thanks to @matteosan1.
In HLTEcal[Hcal]PFClusterIsolationProducer.cc, the parameters recoEcalCandidateProducer is replaced by recoCandidateProducer: @Martin-Grunewald and @perrotta, will it cause a problem ?

[cmsbuild]

A new Pull Request was created by @HuguesBrun (Hugues BRUN) for CMSSW_7_3_X.

PF cluster isolation for muons + TimeCleaned RecHit producer.

It involves the following packages:

DataFormats/RecoCandidate
HLTrigger/Muon
RecoEgamma/EgammaHLTProducers
RecoMuon/L3MuonIsolationProducer

@perrotta, @cmsbuild, @nclopezo, @fwyzard, @Martin-Grunewald, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks.
@battibass, @abbiendi, @lgray, @bellan, @trocino, @bachtis, @rociovilar this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[Martin-Grunewald]

This looks like a verbatim copy of the HLTElectronGenericFilter.
Thus is is better to make that filter templated and
instantiate two instances, one for the Electrons, one for the Muons.

[HuguesBrun]

Hi Martin,
You are right, I will do that

[Martin-Grunewald]

-1
No code duplications!

[Martin-Grunewald]

Yes as it will require a concurrent migration of the HLT menus.

There are five instances of PFClusterIsolationProducers in the
HLT menu, I assume the migration is just moving the InputTag
value from the old recoEcalCandidateProducer parameter to the new
recoCandidateProducer parameter?

[HuguesBrun]

yes exactly.

[slava77]

(need to understand if AssoMap is needed; comments on the producer to follow)

[slava77]

I will not make any further review until the -1 from HLT is cleared

[slava77]

-1

to take of our list, since the pending action item raised by Martin

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6266/895/summary.html

[slava77]

+1

for #6266 dc29ac0
tested in CMSSW_7_3_X_2014-11-14-0200 (test area sign456).
Tests included only the jenkins-like brief versions, which were showing issues in HLT values before.
There are no diffs in performance anymore.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes (tests are also fine). This pull request will be automatically merged.