New code to allow passing context to policy delegate methods: bug 54031
Created attachment 81765 [details]
Comment on attachment 81765 [details]
Just found r77974 causes problems with the page loading, probably we will need to do more things in this patch to make it work.
What's the status on this issue?
(In reply to comment #3)
> What's the status on this issue?
We still need to check the policy handling and fix the problem that causes the problem loading, not sure but could be an easy fix, just some method not implemented of no correctly doing what it should. You can check that applying the patch compilations works but the rendering does not happen and reverting the patch you get the rendering correctly.
I had to check this issue now because after the last patches we can not easily avoid it.
Apparently decidePolicyForNavigationAction message does not work, for some reason, sendSync returns false. I've checked that discarding the return value and passing PolicyUse directly to the frame everything works. Not sure how to to test the message issues, adding people to the CC that probably know how to do it.
I think I found the issue, we have to implement the encode/decodeResourceRequest and resourceResponse.
Created attachment 85042 [details]
Added dummy encoders just so make it work and added a bug to implement the proper support: bug 55934
Thanks for the review Matin, landed http://trac.webkit.org/changeset/80571