Summary: | Calling WebInspectorClient::highlight() during a fade-out animation of the PageOverlay won't stop its animation | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Antoine Quint <graouts> | ||||||
Component: | New Bugs | Assignee: | Antoine Quint <graouts> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | graouts, thorton, timothy, webkit-bug-importer, webkit.review.bot | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Antoine Quint
2013-03-13 11:09:28 PDT
Created attachment 192949 [details]
Patch
Comment on attachment 192949 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=192949&action=review > Source/WebKit2/WebProcess/WebPage/PageOverlay.cpp:158 > + m_fadeAnimationTimer.stop(); Should you reset m_fractionFadedIn to ... something? (In reply to comment #3) > (From update of attachment 192949 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=192949&action=review > > > Source/WebKit2/WebProcess/WebPage/PageOverlay.cpp:158 > > + m_fadeAnimationTimer.stop(); > > Should you reset m_fractionFadedIn to ... something? m_fractionFadedIn needs to be reset to 1.0 I would say. Created attachment 192951 [details]
Patch for landing
Comment on attachment 192951 [details] Patch for landing Clearing flags on attachment: 192951 Committed r145731: <http://trac.webkit.org/changeset/145731> All reviewed patches have been landed. Closing bug. |