Bug 94341 – [EFL][Qt] New inspector/timeline/timeline-decode-resize.html fails with timeout

• Component: Tools / Tests
• Status: RESOLVED INVALID

[Sudarsana Nagineni (babu)]

inspector/timeline/timeline-decode-resize.html was introduced in r125790, but fails on EFL and Qt bots.

EFL: http://build.webkit.org/results/EFL%20Linux%2064-bit%20Debug/r125881%20%284475%29/inspector/timeline/timeline-decode-resize-diff.txt
QT:http://build.webkit.org/results/Qt%20Linux%20Release/r125881%20%2850945%29/inspector/timeline/timeline-decode-resize-diff.txt

[Sudarsana Nagineni (babu)]

Skipping test for EFL in bug 94344.

[Csaba Osztrogonác]

It is skipped on Qt too, please unskip it with the proper fix.

[Zoltan Horvath]

This test times out because of missing libwebp library.

After installing it, the timeout becomes a failure:


--- /home/zoltan/prog/local/WebKit/WebKitBuild/Release/layout-test-results/inspector/timeline/timeline-decode-resize-expected.txt
+++ /home/zoltan/prog/local/WebKit/WebKitBuild/Release/layout-test-results/inspector/timeline/timeline-decode-resize-actual.txt
@@ -14,19 +14,6 @@
     type : "DecodeImage"
     usedHeapSize : <number>
 }
-ResizeImage Properties:
-{
-    children : <object>
-    counters : <object>
-    data : {
-        cached : true
-    }
-    endTime : <number>
-    stackTrace : <object>
-    startTime : <number>
-    type : "ResizeImage"
-    usedHeapSize : <number>
-}
 DecodeImage Properties:
 {
     children : <object>
@@ -38,19 +25,6 @@
     stackTrace : <object>
     startTime : <number>
     type : "DecodeImage"
-    usedHeapSize : <number>
-}
-ResizeImage Properties:
-{
-    children : <object>
-    counters : <object>
-    data : {
-        cached : true
-    }
-    endTime : <number>
-    stackTrace : <object>
-    startTime : <number>
-    type : "ResizeImage"
     usedHeapSize : <number>
 }
 DecodeImage Properties:
@@ -66,19 +40,6 @@
     type : "DecodeImage"
     usedHeapSize : <number>
 }
-ResizeImage Properties:
-{
-    children : <object>
-    counters : <object>
-    data : {
-        cached : true
-    }
-    endTime : <number>
-    stackTrace : <object>
-    startTime : <number>
-    type : "ResizeImage"
-    usedHeapSize : <number>
-}
 DecodeImage Properties:
 {
     children : <object>
@@ -90,19 +51,6 @@
     stackTrace : <object>
     startTime : <number>
     type : "DecodeImage"
-    usedHeapSize : <number>
-}
-ResizeImage Properties:
-{
-    children : <object>
-    counters : <object>
-    data : {
-        cached : true
-    }
-    endTime : <number>
-    stackTrace : <object>
-    startTime : <number>
-    type : "ResizeImage"
     usedHeapSize : <number>
 }
 DecodeImage Properties:
@@ -118,19 +66,6 @@
     type : "DecodeImage"
     usedHeapSize : <number>
 }
-ResizeImage Properties:
-{
-    children : <object>
-    counters : <object>
-    data : {
-        cached : true
-    }
-    endTime : <number>
-    stackTrace : <object>
-    startTime : <number>
-    type : "ResizeImage"
-    usedHeapSize : <number>
-}
 DecodeImage Properties:
 {
     children : <object>
@@ -142,19 +77,6 @@
     stackTrace : <object>
     startTime : <number>
     type : "DecodeImage"
-    usedHeapSize : <number>
-}
-ResizeImage Properties:
-{
-    children : <object>
-    counters : <object>
-    data : {
-        cached : true
-    }
-    endTime : <number>
-    stackTrace : <object>
-    startTime : <number>
-    type : "ResizeImage"
     usedHeapSize : <number>
 }
 DecodeImage Properties:
@@ -170,17 +92,4 @@
     type : "DecodeImage"
     usedHeapSize : <number>
 }
-ResizeImage Properties:
-{
-    children : <object>
-    counters : <object>
-    data : {
-        cached : false
-    }
-    endTime : <number>
-    stackTrace : <object>
-    startTime : <number>
-    type : "ResizeImage"
-    usedHeapSize : <number>
-}

[Zoltan Horvath]

The problem is with this thing that the image resizing/scaling is happening at drawtime in Qt port, so we don't have a separate phase (to measure) for resizing the image. 

You can check it in: Source/WebCore/platform/graphics/qt/ImageQt.cpp void BitmapImage::draw(...)

[Brian Burg]

The related test(s) have been removed from trunk.