New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 778784 link

Starred by 3 users
Status: Archived
Owner:
caseq@chromium.org
Closed: Dec 5
Cc:
█ chenwilliam@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

http/tests/devtools/tracing/decode-resize.html is flaky on all platforms

Project Member Reported by pdr@chromium.org, Oct 26 2017 
http/tests/devtools/tracing/decode-resize.html is flaky:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Fdevtools%2Ftracing%2Fdecode-resize.html&testType=webkit_layout_tests

The failure is:
+CONSOLE ERROR: line 57: Uncaught TypeError: Cannot read property 'dispatchMessage' of undefined

This flakiness seems to have started between r511719 and r511730 but there is nothing in this range that looks likely, other than the devtools test migration:
https://chromium.googlesource.com/chromium/src/+log/05b2d4f3fd83e155c35be3020e33ebbab664c623%5E..b8a6fd5abbbdb8471f49672b7ea71caa378328aa?pretty=fuller&n=

I think what might have happened is that chenwilliam's moving of tests exposed flakiness that was masked by the specific order tests were run in before the migration.

chenwilliam, can you look into this? I will temporarily disable this test.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 26 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9d1798281dc845af4ec86c1c185925f86a44f0bc

commit 9d1798281dc845af4ec86c1c185925f86a44f0bc
Author: Philip Rogers <pdr@chromium.org>
Date: Thu Oct 26 20:30:01 2017

Suppress devtools/tracing/decode-resize.html failures

devtools/tracing/decode-resize.html recently became flaky
(see:  https://crbug.com/778784 ) and this patch temporarily
suppresses this flakiness.

TBR=chrishtr@chromium.org

Bug:  778784 
Change-Id: Ib7681c77ed25cdcd40d9440adc78dc9d6d1984c1
Reviewed-on: https://chromium-review.googlesource.com/740296
Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#511946}
[modify] https://crrev.com/9d1798281dc845af4ec86c1c185925f86a44f0bc/third_party/WebKit/LayoutTests/TestExpectations

Comment 2 by chenwilliam@chromium.org, Nov 14 2017

Cc: chenwilliam@chromium.org
Owner: caseq@chromium.org
The major flakiness issue has now been fixed by:
* https://chromium-review.googlesource.com/c/chromium/src/+/761957

There seems to be a bit of flakiness remaining due to a test specific issue - @caseq, can you take a look:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http/tests/devtools/tracing/decode-resize.html&testType=webkit_layout_tests

example:
https://storage.googleapis.com/chromium-layout-test-archives/Mac10_11_Tests/20313/layout-test-results/http/tests/devtools/tracing/decode-resize-diff.txt
--- /b/s/w/io9lIWWh/layout-test-results/http/tests/devtools/tracing/decode-resize-expected.txt
+++ /b/s/w/io9lIWWh/layout-test-results/http/tests/devtools/tracing/decode-resize-actual.txt
@@ -13,19 +13,10 @@
 imageURL: .../devtools/tracing/resources/test.bmp
 backendNodeId: present
 event: Decode Image
-imageURL: .../devtools/tracing/resources/test.bmp?background
-backendNodeId: present
-event: Decode Image
-imageURL: .../devtools/tracing/resources/test.bmp?border
-backendNodeId: present
-event: Decode Image
 imageURL: .../devtools/tracing/resources/test.gif
 backendNodeId: present
 event: Decode Image
 imageURL: .../devtools/tracing/resources/test.gif?background
-backendNodeId: present
-event: Decode Image
-imageURL: .../devtools/tracing/resources/test.gif?border
 backendNodeId: present
 event: Decode Image
 imageURL: .../devtools/tracing/resources/test.ico
@@ -57,10 +48,4 @@
 event: Decode Image
 imageURL: .../devtools/tracing/resources/test.webp
 backendNodeId: present
-event: Decode Image
-imageURL: .../devtools/tracing/resources/test.webp?background
-backendNodeId: present
-event: Decode Image
-imageURL: .../devtools/tracing/resources/test.webp?border
-backendNodeId: present

Comment 3 by pfeldman@chromium.org, Dec 5

Status: Archived (was: Assigned)
Test failures are processed as a part of a dedicated triage, bulk-closing the bugs.
►

Sign in to add a comment