Bug 54076 – [GTK] Add new files to compilation after r77974

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Alejandro G. Castro]

New code to allow passing context to policy delegate methods: bug 54031

[Alejandro G. Castro]

Created attachment 81765 [details]
Proposed patch

[Alejandro G. Castro]

Comment on attachment 81765 [details]
Proposed patch

Just found r77974 causes problems with the page loading, probably we will need to do more things in this patch to make it work.

[Martin Robinson]

What's the status on this issue?

[Alejandro G. Castro]

(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.

[Alejandro G. Castro]

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.

[Alejandro G. Castro]

I think I found the issue, we have to implement the encode/decodeResourceRequest and resourceResponse.

[Alejandro G. Castro]

Created attachment 85042 [details]
Proposed patch

Added dummy encoders just so make it work and added a bug to implement the proper support: bug 55934

[Alejandro G. Castro]

Thanks for the review Matin, landed http://trac.webkit.org/changeset/80571