New issue
Advanced search Search tips

Issue 860101 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 846920
Owner:
Closed: Jul 6
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Flaky test: http/tests/devtools/tracing/worker-events.js

Project Member Reported by shimazu@chromium.org, Jul 4

Issue description

http/tests/devtools/tracing/worker-events.js
virtual/threaded/http/tests/devtools/tracing/worker-events.js
These two tests are failing very frequently on Mac10.13 (dbg), and sometimes on Linux/Win (dbg) and ASAN bot.

https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=http%2Ftests%2Fdevtools%2Ftracing%2Fworker-events.js


This is the output when failing.
https://test-results.appspot.com/data/layout_results/Mac10_13_Tests__dbg_/3789/layout-test-results/results.html

Here is a copy from the results.html.
====
Tests that worker events are recorded with proper devtools metadata events.

error: TypeError: Cannot read property 'tracingBufferUsage' of null TypeError: Cannot read property 'tracingBufferUsage' of null
    at SDK.TracingManager._bufferUsage (file:///b/s/w/ir/out/Debug/resources/inspector/shell.js:4802:96)
    at SDK.TracingDispatcher.bufferUsage (file:///b/s/w/ir/out/Debug/resources/inspector/shell.js:4812:64)
    at Protocol.InspectorBackend._DispatcherPrototype.dispatch (file:///b/s/w/ir/out/Debug/resources/inspector/shell.js:3380:26)
    at SDK.Target._onMessage (file:///b/s/w/ir/out/Debug/resources/inspector/shell.js:3333:31)
    at SDK.MainConnection._dispatchMessage (file:///b/s/w/ir/out/Debug/resources/inspector/shell.js:3470:41)
    at Common.Object.dispatchEventToListeners (file:///b/s/w/ir/out/Debug/resources/inspector/shell.js:490:23)
    at innerDispatch (file:///b/s/w/ir/out/Debug/resources/inspector/shell.js:967:98)
    at Host.InspectorFrontendAPIImpl._dispatch (file:///b/s/w/ir/out/Debug/resources/inspector/shell.js:967:1)
    at DevToolsAPIImpl._dispatchOnInspectorFrontendAPI (chrome-devtools://devtools/bundled/devtools_compatibility.js:57:36)
    at DevToolsAPIImpl.dispatchMessage (chrome-devtools://devtools/bundled/devtools_compatibility.js:134:12)
Got DevTools worker metadata event(1): TracingSessionIdForWorker
Got DevTools worker metadata event(2): TracingSessionIdForWorker

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 4

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

commit 3bb6c29cc2d9e0083f6ca9bba05caa3f31354466
Author: Makoto Shimazu <shimazu@chromium.org>
Date: Wed Jul 04 01:25:43 2018

Mark http/tests/devtools/tracing/worker-events.js as flaky on Mac10.13

TBR=chenwilliam@chromium.org
NOTRY=true

Bug:  860101 
Change-Id: Ia4c9179dc2c287123c6781c57d6db7c1b4da6393
Reviewed-on: https://chromium-review.googlesource.com/1124734
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Commit-Queue: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572433}
[modify] https://crrev.com/3bb6c29cc2d9e0083f6ca9bba05caa3f31354466/third_party/WebKit/LayoutTests/TestExpectations

Labels: -Sheriff-Chromium
Let me remove sheriff flag since it's marked as flaky.
Mergedinto: 846920
Owner: alph@chromium.org
Status: Duplicate (was: Available)
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 6

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

commit 95a1fb174c2701e8a24dd71943599e559b10d701
Author: Alexei Filippov <alph@chromium.org>
Date: Fri Jul 06 22:40:52 2018

Unmark http/tests/devtools/tracing/worker-events.js as flaky

NOTRY=true
TBR=shimazu@chromium.org
BUG= 860101 , 846920 

Change-Id: Ied88e402a63631cb6c67225f79ca990e3b8a3628
Reviewed-on: https://chromium-review.googlesource.com/1128370
Reviewed-by: Alexei Filippov <alph@chromium.org>
Commit-Queue: Alexei Filippov <alph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573097}
[modify] https://crrev.com/95a1fb174c2701e8a24dd71943599e559b10d701/third_party/WebKit/LayoutTests/TestExpectations

Sign in to add a comment