| Summary: | [WinCairo] Crash in DefaultPolicyDelegate::decidePolicyForNavigationAction. | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | peavo | ||||
| Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> | ||||
| Status: | NEW --- | ||||||
| Severity: | Normal | CC: | achristensen, bfulgham, ossy | ||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
peavo
2015-03-02 05:23:19 PST
Created attachment 247660 [details]
Patch
Comment on attachment 247660 [details]
Patch
Delegate callbacks should never be called with a null web view. I think you should try to figure out why it is null instead of adding a null-check.
(In reply to comment #2) > Comment on attachment 247660 [details] > Patch > > Delegate callbacks should never be called with a null web view. I think you > should try to figure out why it is null instead of adding a null-check. Ok, thanks for reviewing :) I will investigate why it happens. Comment on attachment 247660 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=247660&action=review > Source/WebKit/win/DefaultPolicyDelegate.cpp:106 > + Even though Anders doesn't like this check and return, it definitely seems like we should ASSERT here (or maybe even RELEASE_ASSERT?) so we can catch when this happens. (In reply to comment #4) > Comment on attachment 247660 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=247660&action=review > > > Source/WebKit/win/DefaultPolicyDelegate.cpp:106 > > + > > Even though Anders doesn't like this check and return, it definitely seems > like we should ASSERT here (or maybe even RELEASE_ASSERT?) so we can catch > when this happens. Thanks :) I will add the ASSERT, and see if I can find the cause. |