Summary: | WTR crashes if an external URL is passed as a parameter | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Zoltan Horvath <zoltan> | ||||||||
Component: | Tools / Tests | Assignee: | Balazs Kelemen <kbalazs> | ||||||||
Status: | RESOLVED WONTFIX | ||||||||||
Severity: | Normal | CC: | allan.jensen, ap, hausmann, kbalazs, kenneth, sam, webkit.review.bot | ||||||||
Priority: | P2 | Keywords: | Qt, QtTriaged | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | Linux | ||||||||||
Bug Depends on: | 90623 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Zoltan Horvath
2012-06-01 07:39:50 PDT
Created attachment 148127 [details]
Patch
Comment on attachment 148127 [details]
Patch
r=me
Comment on attachment 148127 [details] Patch Clearing flags on attachment: 148127 Committed r121899: <http://trac.webkit.org/changeset/121899> All reviewed patches have been landed. Closing bug. Re-opened since this is blocked by 90623 Created attachment 151693 [details]
Patch
Could anybody review this? It's there for weeks. If not I will go to the quick route and fix it in wtr to avoid touching API but still I believe the API is not perfect here. Sam: Ping :) (In reply to comment #7) > Could anybody review this? It's there for weeks. If not I will go to the quick route and fix it in wtr to avoid touching API but still I believe the API is not perfect here. I realized that there is no need for this since this crash have been disappeared because external urls are now allowed in WTR/DRT. So only the API patch makes sense, if we agree that it's an improvement. |