Issue metadata
Sign in to add a comment
|
FrameImplTest.PostMessage is flaky |
||||||||||||||||||||||
Issue descriptionFrameImplTest.PostMessage is flaky. Findit has detected 3 flake occurrences of this test within the past 24 hours. List of all flake occurrences can be found at: https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSBUZsYWtlIjljaHJvbWl1bUB3ZWJydW5uZXJfYnJvd3NlcnRlc3RzQEZyYW1lSW1wbFRlc3QuUG9zdE1lc3NhZ2UM. Unless the culprit CL is found and reverted, please disable this test first within 30 minutes then find an appropriate owner. If the result above is wrong, please file a bug using this link: https://bugs.chromium.org/p/chromium/issues/entry?status=Unconfirmed&labels=Pri-1,Test-Findit-Wrong&components=Tools%3ETest%3EFindit%3EFlakiness&summary=%5BFindit%5D%20Flake%20Detection%20-%20Wrong%20result%20for%20FrameImplTest.PostMessage&comment=Link%20to%20flake%20occurrences%3A%20https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSBUZsYWtlIjljaHJvbWl1bUB3ZWJydW5uZXJfYnJvd3NlcnRlc3RzQEZyYW1lSW1wbFRlc3QuUG9zdE1lc3NhZ2UM Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
,
Nov 22
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/941a0d7416a1dc9530acdedab00bf05611f1ce22 commit 941a0d7416a1dc9530acdedab00bf05611f1ce22 Author: Hayato Ito <hayato@chromium.org> Date: Thu Nov 22 04:11:52 2018 Disable flaky FrameImplTest.PostMessage test Chromium sheriff. TBR=wez@chromium.org Bug: 907717 Change-Id: Id0a5bb81ed623572e87f19b9229048febdd3ff2c Reviewed-on: https://chromium-review.googlesource.com/c/1347642 Reviewed-by: Hayato Ito <hayato@chromium.org> Commit-Queue: Hayato Ito <hayato@chromium.org> Cr-Commit-Position: refs/heads/master@{#610299} [modify] https://crrev.com/941a0d7416a1dc9530acdedab00bf05611f1ce22/webrunner/browser/frame_impl_browsertest.cc
,
Nov 22
Disabled. Let me remove Sheriff-Chromium label. wez@, "git cl" suggested you as an owner of the test. Could you take a look, or re-route to the right owner?
,
Nov 23
,
Nov 27
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d8c78a28251a4fa7fef218d47b83cfad39dca314 commit d8c78a28251a4fa7fef218d47b83cfad39dca314 Author: Wez <wez@chromium.org> Date: Tue Nov 27 18:37:40 2018 Revert "Disable all tests in FrameImplTest." This reverts commit 912050e157c4fd552a19a44d87cbd67e42d38411. Reason for revert: Fuchsia SDK believed to be the cause of these failures; speculatively reverting to re-enable the tests to evaluate. Original change's description: > Disable all tests in FrameImplTest. > > Almost all have been reported as flaky. Disable all to avoid having > reports coming in from whichever subtest was not reported yet. Caused a > lot of red jobs on the Fuchsia x64 bot. > > TBR=wez@chromium.org > > Bug: 907804 > Change-Id: I2fb4da79bf1e4251e14900ef4cabb890b7b3380d > Reviewed-on: https://chromium-review.googlesource.com/c/1349315 > Reviewed-by: Rune Lillesveen <futhark@chromium.org> > Commit-Queue: Rune Lillesveen <futhark@chromium.org> > Cr-Commit-Position: refs/heads/master@{#610580} TBR=wez@chromium.org,futhark@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 907804 , 907859 , 907717 Change-Id: I72103faff6457a93136925afdab9d7f003061369 Reviewed-on: https://chromium-review.googlesource.com/c/1351100 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by: Rune Lillesveen <futhark@chromium.org> Reviewed-by: Fabrice de Gans-Riberi <fdegans@chromium.org> Reviewed-by: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#611212} [modify] https://crrev.com/d8c78a28251a4fa7fef218d47b83cfad39dca314/webrunner/browser/frame_impl_browsertest.cc [modify] https://crrev.com/d8c78a28251a4fa7fef218d47b83cfad39dca314/webrunner/renderer/on_load_script_injector.h |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by hayato@chromium.org
, Nov 22