Bug 58848

Summary: [Qt] REGRESSION(84057): It made 4 API tests fail
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: Tools / TestsAssignee: Caio Marcelo de Oliveira Filho <cmarcelo>
Severity: Critical CC: cmarcelo, commit-queue, kenneth, kling, luiz
Priority: P1 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 59070    
Bug Blocks: 46968    
Description Flags
patch none

Description Csaba Osztrogonác 2011-04-18 16:08:03 PDT
FAIL!  : tst_QWebPage::backActionUpdate() Compared values are not the same
   Actual (loadSpy.count()): 1
   Expected (2): 2
   Loc: [/home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/Source/WebKit/qt/tests/qwebpage/tst_qwebpage.cpp(1453)]

FAIL!  : tst_QWebPage::errorPageExtensionInFrameset() Received a fatal error.
   Loc: [Unknown file(0)]

FAIL!  : tst_QWebFrame::horizontalScrollAfterBack() '(frame->scrollBarGeometry(Qt::Horizontal)).height()' returned FALSE. ()
   Loc: [/home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/Source/WebKit/qt/tests/qwebframe/tst_qwebframe.cpp(3152)]

FAIL!  : tst_QWebFrame::requestedUrl() Compared values are not the same
   Actual (spy.count()): 2
   Expected (3): 3
   Loc: [/home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/Source/WebKit/qt/tests/qwebframe/tst_qwebframe.cpp(2404)]
Comment 1 Csaba Osztrogonác 2011-04-18 16:10:16 PDT
Hey men, you should have checked the buildbot after landing a patch. :-/
Comment 2 Caio Marcelo de Oliveira Filho 2011-04-20 23:21:52 PDT
Created attachment 90499 [details]

This patch should fix 3 tests failing. The requestedUrl() one is handled by bug 59070.
Comment 3 Csaba Osztrogonác 2011-04-20 23:36:58 PDT
Comment on attachment 90499 [details]

LGTM, r=me
Comment 4 WebKit Commit Bot 2011-04-21 00:29:51 PDT
Comment on attachment 90499 [details]

Clearing flags on attachment: 90499

Committed r84475: <http://trac.webkit.org/changeset/84475>
Comment 5 WebKit Commit Bot 2011-04-21 00:29:56 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Caio Marcelo de Oliveira Filho 2011-04-21 00:35:25 PDT
Reopening since there is still a dependency bug opened to fix the remaining test case.