New issue
Advanced search Search tips

Issue 834829 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug-Regression



Sign in to add a comment

smoothness.key_silk_cases/http://www.google.com/#q=google in smoothness.key_silk_cases failing on multiple builders

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Apr 19 2018

Issue description

Comment 1 by eyaich@chromium.org, Apr 19 2018

Cc: -eyaich@google.com eyaich@chromium.org
Components: Speed>Benchmarks>Waterfall
Labels: OS-Android Type-Bug-Regression
Owner: ajuma@chromium.org
Looks like a pretty clear failure range on android.  Going to kick off a bisect.

https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=thread_times.key_silk_cases%2Fhttp%3A%2F%2Fwww.google.com%2F%23q%3Dgoogle&testType=thread_times.key_silk_cases

Comment 3 by eyaich@chromium.org, Apr 19 2018

Link to recent swarming task here: 
https://chromium-swarm.appspot.com/task?id=3cef2eaab79c7210&refresh=10&show_raw=1

Log snippet: 
UtLy8.png
(INFO) 2018-04-18 01:38:35,189 cmd_helper._ValidateAndLogCommand:160  [host]> /b/swarming/w/ir/third_party/catapult/devil/bin/deps/linux2/x86_64/bin/adb -s 01e40655c29b4f29 shell 'rm -f /data/local/tmp/temp_file-11002a2457cfb.png'
(WARNING) 2018-04-18 01:38:35,203 artifact_results.AddArtifact:128  Moving artifact file '/b/swarming/w/itDMRiQ_/tmpMUtLy8.png' to '/b/swarming/w/itDMRiQ_/tmpEtVxmatelemetry/artifacts'
Traceback (most recent call last):
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/internal/story_runner.py", line 123, 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 313, 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/measurements/smoothness.py", line 76, in ValidateAndMeasurePage
    metric.AddResults(model, renderer_thread, records, results)
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/web_perf/metrics/smoothness.py", line 54, in AddResults
    self.VerifyNonOverlappedRecords(interaction_records)
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/web_perf/metrics/timeline_based_metric.py", line 84, in VerifyNonOverlappedRecords
    if _TimeRangesHasOverlap(((r.start, r.end) for r in interaction_records)):
  File "/b/swarming/w/ir/third_party/catapult/telemetry/telemetry/web_perf/metrics/timeline_based_metric.py", line 19, in _TimeRangesHasOverlap
    last_range = sorted_time_ranges[0]
IndexError: list index out of range

Comment 4 by eyaich@chromium.org, Apr 19 2018

Potential  this might be the same culprit as  crbug.com/834833  given they are failing at the same build witht he same strack trace.
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Apr 19 2018

Cc: mdjones@chromium.org jinsuk...@chromium.org tiborg@chromium.org
Owner: tiborg@chromium.org
Status: Assigned (was: Available)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/14e34581c40000

Update top offset of motion events on hover in CompositorViewHolder by tiborg@chromium.org
https://chromium.googlesource.com/chromium/src/+/0ef4af14dff5f447bc8746e7aa1634c3adcdd3bb

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 6 by tiborg@chromium.org, Apr 19 2018

Status: Fixed (was: Assigned)
Change 0ef4af14dff5f447bc8746e7aa1634c3adcdd3bb got reverted. Closing this.

Sign in to add a comment