Bug 98136

Summary: [Qt][WK2] inspector/extensions/extensions-network.html times out
Product: WebKit Reporter: Michael Brüning <michael.bruning>
Component: WebKit QtAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: andersca, michael.bruning, ossy, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Michael Brüning 2012-10-02 03:28:28 PDT 
This test times out on Release builds and crashes/asserts on Debug builds. 

This is caused because WebPageProxy* WebInspectorProxy::platformCreateInspectorPage() is not implemented, so it will always fail until that is changed.
Comment 1 Csaba Osztrogonác 2012-10-02 03:33:23 PDT 
Unfortunately this test is skipped because of another bug - https://bugs.webkit.org/show_bug.cgi?id=89349

But you can add it to qt-5.0-wk2/TestExpectations too.
Comment 2 Michael Brüning 2012-10-02 03:36:08 PDT 
Created attachment 166654 [details]
Patch
Comment 3 WebKit Review Bot 2012-10-02 03:39:37 PDT 
Attachment 166654 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/plat..." exit_code: 1
LayoutTests/platform/qt-5.0-wk2/TestExpectations:1:  No port uses path LayoutTests/platform/qt-5.0-wk2/TestExpectations for test_expectations  [test/expectations] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Csaba Osztrogonác 2012-10-02 03:41:24 PDT 
It seems you have to commit it manually because of buggy check-webkit-style
Comment 5 Csaba Osztrogonác 2012-10-02 03:46:50 PDT 
(In reply to comment #4)
> It seems you have to commit it manually because of buggy check-webkit-style

I filed a new bug report against check-webkit-style:
https://bugs.webkit.org/show_bug.cgi?id=98140
Comment 6 Michael Brüning 2012-10-02 04:02:45 PDT 
(In reply to comment #4)
> It seems you have to commit it manually because of buggy check-webkit-style

Okay, we'll just have to wait until I have my special committer super powers back then...
Comment 7 Michael Brüning 2012-10-02 08:43:24 PDT 
Committed r130173: <http://trac.webkit.org/changeset/130173>
Comment 8 Csaba Osztrogonác 2012-10-02 08:46:54 PDT 
The bug isn't fixed, only the skipping landed.