Bug 82328 - [GTK] Test /webkit/webview/icon-uri times out in GTK Linux 64-bit Release bot
Summary: [GTK] Test /webkit/webview/icon-uri times out in GTK Linux 64-bit Release bot
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk
Depends on:
Blocks: 74717
  Show dependency tree / graph
 
Reported: 2012-03-27 06:13 PDT by Carlos Garcia Campos
Modified: 2014-01-17 06:00 PST (History)
2 users (show)

See Also:

Attachments
Patch (1.73 KB, patch)
2014-01-17 04:35 PST, Alberto Garcia 		cgarcia: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Garcia Campos 2012-03-27 06:13:53 PDT 
TEST: ./WebKitBuild/Release/Programs/unittests/testwebview... (pid=16952)
  /webkit/webview/icon-uri:                                            
** ERROR **: Didn't get icon-uri before timing out.
FAIL
Comment 1 Simon Pena 2013-07-23 07:40:21 PDT 
I just run this with 

Tools/jhbuild/jhbuild-wrapper --gtk run WebKitBuild/{Release,Debug}/Programs/unittests/testwebview

and both configurations finish without timing out. Is there any policy to unskip these tests?
Comment 2 Alberto Garcia 2014-01-16 05:51:54 PST 
This also works fine for me as well, there must be something wrong in the bot.
Comment 3 Alberto Garcia 2014-01-17 04:35:56 PST 
Created attachment 221456 [details]
Patch

Yeah, the icon database in ~/.local/share/webkit/icondatabase/WebpageIcons.db
in the buildbot was somehow malformed:

sqlite> .databases
Error: database disk image is malformed

I removed it and now the test works fine.
Comment 4 Carlos Garcia Campos 2014-01-17 05:57:54 PST 
Comment on attachment 221456 [details]
Patch

I think the test should use a new temporal database in /tmp every time, but I guess it can be done in a different patch, let's unskip this for now.
Comment 5 Alberto Garcia 2014-01-17 06:00:04 PST 
Committed r162199: <http://trac.webkit.org/changeset/162199>