Bug 56946

Summary:

[Qt][Symbian] Fix Api test failure -- tst_QWebFrame::setHtmlWithBaseURL

Product:

WebKit

Reporter:

Yi Shen <max.hong.shen>

Component:

WebKit Qt

Assignee:

Yi Shen <max.hong.shen>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ademar, commit-queue, laszlo.gombos, menard, s.mathur

Priority:

Version:

528+ (Nightly build)

Hardware:

S60 Hardware

OS:

Other

Bug Depends on:

Bug Blocks:

38654, 50925

Attachments:

Description	Flags
first try	none
proposed fix	none
updated with Siddharth's suggestion	none
fix the changlog	none

Description Yi Shen 2011-03-23 12:11:21 PDT

tst_QWebFrame::setHtmlWithBaseURL tries to load an image resource from a particular path, while, this resource doesn't exist under symbian environment. Will provide a patch soon.

Comment 1 Yi Shen 2011-03-23 12:14:43 PDT

Created attachment 86661 [details]
first try

Comment 2 Yi Shen 2011-03-24 07:42:42 PDT

Created attachment 86772 [details]
proposed fix

Comment 3 Kenneth Rohde Christiansen 2011-03-24 11:42:46 PDT

Comment on attachment 86772 [details]
proposed fix

Why dont other platforms need this?

Comment 4 Yi Shen 2011-03-25 11:43:23 PDT

(In reply to comment #3)
> (From update of attachment 86772 [details])
> Why dont other platforms need this?

For desktop env like Linux, Windows, they don't need this. For other mobile env like Meego, Maemo, they may need to add a similar, platform-dependent patch. It would be good to be done in a separated bug. Thanks!

Comment 5 Siddharth Mathur 2011-03-26 13:56:13 PDT

(In reply to comment #2)
> Created an attachment (id=86772) [details]
> proposed fix

If the .png and other content can be loaded from a qrc:// URL as part of the test, then we should prefer that. I.e add to qmake's RESOURCES list. 

If the test requires loading a file from local disk, then we should install the .png within c:\private\UID3ofTestRunnerEXE\ , and not in C:\resource

Comment 6 Yi Shen 2011-04-05 09:50:08 PDT

Created attachment 88258 [details]
updated with Siddharth's suggestion

Comment 7 Yi Shen 2011-04-05 09:53:29 PDT

Created attachment 88261 [details]
fix the changlog

Comment 8 Tor Arne Vestbø 2011-04-26 15:43:37 PDT

Comment on attachment 88261 [details]
fix the changlog

lgtm

Comment 9 WebKit Commit Bot 2011-04-26 16:19:04 PDT

Comment on attachment 88261 [details]
fix the changlog

Clearing flags on attachment: 88261

Committed r84974: <http://trac.webkit.org/changeset/84974>

Comment 10 WebKit Commit Bot 2011-04-26 16:19:09 PDT

All reviewed patches have been landed.  Closing bug.