Summary: | [WK2] Make sure m_iconDatabaseImpl is not null before closing it | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nick Diego Yamane (diegoyam) <nick.diego> | ||||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, sergio, simon.fraser | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Nick Diego Yamane (diegoyam)
2013-06-24 13:19:05 PDT
Created attachment 205321 [details]
Patch
Is this a crash observed in the field, or when running tests? Since r150474 WebIconDatabase is closed unconditionally in TestController destructor, in that case WebKitTestRunner crashes when no icon database path is set. This misbehavior was observed in ports such as efl and nix. Created attachment 205333 [details]
Patch
proposed patch
Comment on attachment 205333 [details] Patch Clearing flags on attachment: 205333 Committed r151932: <http://trac.webkit.org/changeset/151932> All reviewed patches have been landed. Closing bug. |