Issue metadata
Sign in to add a comment
|
FrameImplTest.ExecuteJavaScriptImmediate is flaky |
||||||||||||||||||||||||
Issue descriptionFrameImplTest.ExecuteJavaScriptImmediate 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=ag9zfmZpbmRpdC1mb3ItbWVyUwsSBUZsYWtlIkhjaHJvbWl1bUB3ZWJydW5uZXJfYnJvd3NlcnRlc3RzQEZyYW1lSW1wbFRlc3QuRXhlY3V0ZUphdmFTY3JpcHRJbW1lZGlhdGUM. 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.ExecuteJavaScriptImmediate&comment=Link%20to%20flake%20occurrences%3A%20https://findit-for-me.appspot.com/flake/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyUwsSBUZsYWtlIkhjaHJvbWl1bUB3ZWJydW5uZXJfYnJvd3NlcnRlc3RzQEZyYW1lSW1wbFRlc3QuRXhlY3V0ZUphdmFTY3JpcHRJbW1lZGlhdGUM Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
,
Nov 22
,
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 bugdroid1@chromium.org
, Nov 22