Fix Target.targetCreated handler for existing targets

[markiz]

Came up in our large system spec suite, Client does not respond to session_id, so I'm assuming target#session_id should be right here. Haven't seen any errors related to that since.

[route]

I guess you are running specs with ferrum flatten: false, any chance to switch to true?

[markiz]

@route

I guess you are running specs with ferrum flatten: false

yeah. I'm afraid I don't remember the exact reason why we had to switch to flatten: false, but it probably had to do with stability.

I guess you are running specs with ferrum flatten: false, any chance to switch to true?

Yeah, checking it out. I'm seeing some performance regressions with flatten (5-6 minutes runs vs 3-4 minutes; the normal deviation for this particular suite is ~1 minute though), and also some intermittent Browser is dead or given window is closed.

Is that going to be the new default or something in v1?

[route]

Interesting, by any chance any reproducible test out of your project, at least for the error, perfomance regressions would be hard to debug, but very interesting.