New issue
Advanced search Search tips

Issue 901493 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Nov 5
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

blink_perf collapsed-border tests hit TracingTimeoutException on Nexus6 webview

Project Member Reported by crouleau@chromium.org, Nov 2

Issue description

https://ci.chromium.org/p/chrome/builders/luci.chrome.ci/Android%20Nexus6%20WebView%20Perf/3109

# Affected Tests:
blink_perf.paint/large-table-background-change-with-visible-collapsed-borders.html
blink_perf.paint/large-table-collapsed-border-change-with-backgrounds.html
blink_perf.paint/large-table-collapsed-border-change-with-text.html


# Error:
Traceback (most recent call last):
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/internal/story_runner.py", line 126, in _RunStoryAndProcessErrorIfNeeded
    state.RunStory(results)
  File "/b/swarming/w/ir/third_party/catapult/common/py_trace_event/py_trace_event/trace_event_impl/decorators.py", line 52, in traced_function
    return func(*args, **kwargs)
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/page/shared_page_state.py", line 302, in RunStory
    self._current_page, self._current_tab, results)
  File "/b/swarming/w/ir/third_party/catapult/common/py_trace_event/py_trace_event/trace_event_impl/decorators.py", line 75, in traced_function
    return func(*args, **kwargs)
  File "/b/swarming/w/ir/tools/perf/benchmarks/blink_perf.py", line 304, in ValidateAndMeasurePage
    trace_data = tab.browser.platform.tracing_controller.StopTracing()[0]
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/core/tracing_controller.py", line 53, in StopTracing
    return self._tracing_controller_backend.StopTracing()
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/internal/platform/tracing_controller_backend.py", line 157, in StopTracing
    '\n'.join(raised_exception_messages))
TracingException: Exceptions raised when trying to stop tracing:
Traceback (most recent call last):
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/internal/platform/tracing_controller_backend.py", line 145, in StopTracing
    agent.CollectAgentTraceData(builder)
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/internal/platform/tracing_agent/chrome_tracing_agent.py", line 248, in CollectAgentTraceData
    '\n'.join(raised_exception_messages))
ChromeTracingStoppedError: Exceptions raised when trying to collect Chrome devtool tracing:
Error when collecting Chrome tracing on devtools at port localabstract:webview_devtools_remote_4902:
Traceback (most recent call last):
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/internal/platform/tracing_agent/chrome_tracing_agent.py", line 237, in CollectAgentTraceData
    client.CollectChromeTracingData(trace_data_builder)
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/devtools_client_backend.py", line 489, in CollectChromeTracingData
    self._tracing_backend.CollectTraceData(trace_data_builder, timeout)
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/tracing_backend.py", line 237, in CollectTraceData
    self._CollectTracingData(trace_data_builder, timeout)
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/internal/backends/chrome_inspector/tracing_backend.py", line 279, in _CollectTracingData
    'the timeout amount.' % elapsed_time)
TracingTimeoutException: Only received partial trace data due to timeout after 120.070789814 seconds. If the trace data is big, you may want to increase the timeout amount.


 
Blocking: 901474
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 2

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

commit 63b1278b6c558845b4c3bf971759fd474addb1b5
Author: Caleb Rouleau <crouleau@chromium.org>
Date: Fri Nov 02 22:03:28 2018

Disable failing blink_perf collapsed-border tests on Nexus6_Webview

NOTRY=true
TBR=nednguyen@google.com
CC=wangxianzhu@chromium.org

Bug:  901493 
Change-Id: Iec05bec384214b2e3aa0fad1ea00687c57636f96
Reviewed-on: https://chromium-review.googlesource.com/c/1316407
Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#605065}
[modify] https://crrev.com/63b1278b6c558845b4c3bf971759fd474addb1b5/tools/perf/expectations.config

Blocking: -901474
Status: WontFix (was: Assigned)
The load of the test may be too heavy for some Android devices. It's fine not to test them on Android.
Okay, see also  issue 799540 .

Since a lot of these tests are failing on Nexus 6 Webview, I will disable this whole benchmark on Nexus6 Webview. Note that we will no longer have Webview coverage for blink_perf.paint after this change.

https://ci.chromium.org/p/chrome/builders/luci.chrome.ci/Android%20Nexus6%20WebView%20Perf/3125

Ah wait, we should still have Nexus5X_Webview coverage :)
Project Member

Comment 7 by bugdroid1@chromium.org, Nov 5

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

commit bf7ad46dee47954df0406a5e87cbe5ba23d403c2
Author: Caleb Rouleau <crouleau@chromium.org>
Date: Mon Nov 05 19:00:43 2018

Disable blink_perf.paint on Nexus6_Webview. Too heavy for old device.

TBR=nednguyen@google.com
NOTRY=true

Bug:  901493 
Change-Id: Idae82ae8f4b325dd4346115853b193429fbbda1d
Reviewed-on: https://chromium-review.googlesource.com/c/1318398
Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#605401}
[modify] https://crrev.com/bf7ad46dee47954df0406a5e87cbe5ba23d403c2/tools/perf/expectations.config

Sign in to add a comment