New issue
Advanced search Search tips

Issue 907736 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 18
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

http/.../elements-save-to-temp-var.js in not_site_per_process_webkit_layout_tests failing on multiple builders

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Nov 22

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of hayato@google.com

http/.../elements-save-to-temp-var.js in not_site_per_process_webkit_layout_tests failing on multiple builders

Builders failed on: 
- linux-xenial-rel: 
  https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-xenial-rel
- Win7 Tests (1): 
  https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win7%20Tests%20%281%29


 
Marking http/tests/devtools/elements/elements-save-to-temp-var.js flaky in https://chromium-review.googlesource.com/c/chromium/src/+/1347959/
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 22

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

commit df2b0c38e0b38222fc014f0c59f9ae1468e84db9
Author: Hayato Ito <hayato@chromium.org>
Date: Thu Nov 22 07:49:51 2018

Mark http/tests/devtools/elements/elements-save-to-temp-var.js flaky

Chromium Sheriff

TBR=luoe@chromium.org

Bug:  907736 
Change-Id: I00571640c46263961369a397ca1fc59066bccd2e
Reviewed-on: https://chromium-review.googlesource.com/c/1347959
Reviewed-by: Hayato Ito <hayato@chromium.org>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610342}
[modify] https://crrev.com/df2b0c38e0b38222fc014f0c59f9ae1468e84db9/third_party/WebKit/LayoutTests/TestExpectations

Labels: -Sheriff-Chromium
Owner: l...@chromium.org
Status: Assigned (was: Available)
Marked as flaky.
Let me remove Sheriff-Chromium label.

luoe@, could you have chance to take a look? or re-route to the right owner?
Thanks for the report.  I believe my drive-by fix in CL [1] addressed the flakiness, as the dashboard shows the test passing now.  I'll update test expectations.

[1] https://chromium.googlesource.com/chromium/src/+/cc2fb34c6c8a513781dc7d78e827f340605681e9
Status: Fixed (was: Assigned)

Sign in to add a comment