EventHandler getters return RefPtr<EventListener> (not v8::Handle<v8::Value>)

EventHandler getters return RefPtr<EventListener> (not v8::Handle<v8::Value>)

For consistency with other DOM objects, per request of haraken@
https://codereview.chromium.org/117063002/#msg7

R=haraken

[Nils Barth (inactive), 2013-12-19 06:01:14]

Not sure correct way to do this;
this just returns the result of imp->attributeName() directly.
Is there anything else involved?

[Nils Barth (inactive), 2013-12-19 07:15:58]

On 2013/12/19 06:01:14, Nils Barth wrote:
> Not sure correct way to do this;
> this just returns the result of imp->attributeName() directly.
> Is there anything else involved?

This breaks many tests (b/c it's not returning the event listener function
correctly);
what should we be returning?
(Should we be using V8EventListenerList::getEventListener ?)

[haraken, 2013-12-19 10:40:25]

On 2013/12/19 07:15:58, Nils Barth wrote:
> On 2013/12/19 06:01:14, Nils Barth wrote:
> > Not sure correct way to do this;
> > this just returns the result of imp->attributeName() directly.
> > Is there anything else involved?
> 
> This breaks many tests (b/c it's not returning the event listener function
> correctly);
> what should we be returning?
> (Should we be using V8EventListenerList::getEventListener ?)

Yes, we must use V8EventListenerList::getEventListener(), otherwise a listener
wrapper won't be prepared.

You also need to change a return type of C++ methods from EventListener* to
PassRefPtr<EventListener>.