Summary: | Web Inspector: Attempting to dismiss a popover that is already being dismissed causes an error | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Matt Baker <mattbaker> | ||||||
Component: | Web Inspector | Assignee: | Matt Baker <mattbaker> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Matt Baker
2016-04-07 20:23:12 PDT
Created attachment 275973 [details]
[Patch] Proposed Fix
Comment on attachment 275973 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=275973&action=review Is the dismissing flag ever cleared when the animation completes? Or do you rely on later present() calls to reset it? It seems weird to have the dismissing flag always true even once it's out of the DOM. > Source/WebInspectorUI/ChangeLog:24 > +2016-04-07 Matt Baker <mattbaker@apple.com> You made a double changelog. Created attachment 275984 [details]
[Patch] Proposed Fix
Comment on attachment 275984 [details] [Patch] Proposed Fix Clearing flags on attachment: 275984 Committed r199227: <http://trac.webkit.org/changeset/199227> All reviewed patches have been landed. Closing bug. |