Avoid shadowing local |event| in CueTimeline::updateActiveCues

Avoid shadowing local |event| in CueTimeline::updateActiveCues

Dispatching to the track element would declare an |event| in its scope
that would shadow an |event| in the parent scope.
Avoid this by moving the event-creation-for-a-target to a small helper
function and then use that for all scheduling of events in this method.

BUG=321654
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190543

[fs, 2015-02-19 15:41:27]

fs@opera.com changed reviewers:
+ philipj@opera.com

[philipj_slow, 2015-02-19 17:36:14]

lgtm, but s/it's/its/ in description

[fs, 2015-02-20 08:28:27]

On 2015/02/19 17:36:14, philipj_UTC7 wrote:
> lgtm, but s/it's/its/ in description

Fixed.

[fs, 2015-02-20 08:28:31]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2015-02-20 08:29:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/935423002/1

[commit-bot: I haz the power, 2015-02-20 08:32:08]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=190543