Bug 9667 – [S60] ResolveLinkL callback never called

• Component: WebKit API
• Status: CLOSED FIXED

[Brian Phillips]

Problem:
ClientResolveURL flag is never being set to true, so the ResolveLinkL call back never happens for resolving urls when the client is using the MBrCtlLinkResolver interface.

Expected Behavior:
Should recieve call back to ResolveLinkL and ResolveEmbeddedLinkL when the MBrCtlLinkResolver interface is used and a link is clicked in the browser.

Version:
Not sure what version, it was a build I did myself off of pretty recent code (maybe a week old max from today June 30th.)  David.Carson at nokia.com was able to confirm the bug.

[Krishna]

Confirmed the status as NEW as David Carson was acknowledged the issue.

[Joseph Ligman]

Created attachment 9836 [details]
Files: WebKitLoader.cpp, BufferTransaction.h, BufferTransaction.cpp

The SetClientResolvesUrl setting was not set when pages were loaded.
If the ResolveLinkL callback failed to handle the link the transaction a crash occured

[Yongjun Zhang]

Comment on attachment 9836 [details]
Files: WebKitLoader.cpp, BufferTransaction.h, BufferTransaction.cpp

1. CancelTransaction() is an callback function, please rename it as CancelTransactioCB().  2. please remove replacing the tabs with blanks, otherwise svn will not accept the patch.

[Joseph Ligman]

Created attachment 9838 [details]
Files: WebKitLoader.cpp, BufferTransaction.h, BufferTransaction.cpp

In TSW Bug ID: VKRR-6PH9VQ 
Changed callback function name to CancelTransactionCB
Removed tabs

Test Case: (source in TSW)
1) Copy the html files to <SDK_Installation drive>:\epoc32\WINSCW\C
2) Compile the application in CodeWarrior
3) Add a break point to the function CProto1BCContainer::ResolveLinkL in the file Proto1BCContainer.cpp
4) Complie and debug the application
5) The application can be found in installed folder\Proto1-1st
6) Upon invokation, the application will show a embedded html page.
7) Click any one of the links

[Yongjun Zhang]

Comment on attachment 9838 [details]
Files: WebKitLoader.cpp, BufferTransaction.h, BufferTransaction.cpp

r=me

[Bradley Morrison]

Applied, r15778.

[Krishna]

Closed

[Joel Parks]

re-purposing InTSW keyword for use by QtWebkit team