[Webkit-unassigned] [Bug 58953] [V8] Use implicit references for V8 listeners on DOM nodes.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Apr 22 18:21:17 PDT 2011
https://bugs.webkit.org/show_bug.cgi?id=58953
--- Comment #9 from Vitaly Repeshko <vitalyr at chromium.org> 2011-04-22 18:21:17 PST ---
(In reply to comment #6)
> (From update of attachment 90797 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=90797&action=review
>
> > Source/WebCore/bindings/v8/V8AbstractEventListener.h:94
> > + v8::Persistent<v8::Object> getExistingListenerObjectPersistentHandle()
>
> Please remove "get" prefix.
Removed.
> > Source/WebCore/dom/EventTarget.cpp:394
> > +EventListenerIterator::EventListenerIterator() : m_index(0) {}
>
> The { and } should each be on their own line.
Fixed.
> > Source/WebCore/dom/EventTarget.cpp:396
> > +EventListenerIterator::EventListenerIterator(EventTarget* target) : m_index(0)
>
> ": m_index(0)" should be on its own line and indented four spaces.
Fixed.
> > Source/WebCore/dom/EventTarget.cpp:410
> > + EventListenerVector& listeners = *m_mapIterator->second;
> > + if (m_index < listeners.size())
> > + return listeners[m_index++].listener.get();
>
> We don't need to reset m_index to zero when we ++m_mapIterator ?
Whoops. Fixed.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list