[Webkit-unassigned] [Bug 60779] Bug in rubber banding logic for scroll animators

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri May 13 11:58:37 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=60779





--- Comment #8 from Jon Lee <jonlee at apple.com>  2011-05-13 11:58:37 PST ---
(In reply to comment #7)

> > Source/WebCore/page/EventHandler.cpp:-2144
> > -    view = m_frame->view();
> > -    if (!view)
> > -        return false;
> > -
> 
> Why this change?

Earlier in the method this check is already performed. Unless there's a possibility that the frame will lose the view by the time we hit this line, I don't think we have to check twice.

--

I am noting all of your renaming suggestions, and will upload a new patch shortly.

-- 
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