Issue metadata
Sign in to add a comment
|
CrElementsSliderTest.All is flaky |
||||||||||||||||||||||||
Issue descriptionFindit identified the culprit r618068 as introducing flaky test(s) summarized in https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vODk1MmM2NzNjYjAwNzBmOGExNjQxNDY3ODhiYzc1ZjMzYTNhOGNjNAw Please revert the culprit or disable the test(s) asap. If you are the owner, please fix! If the culprit 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%20Analyzer%20-%20Wrong%20culprit%20r618068&comment=Link%20to%20Culprit%3A%20https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vODk1MmM2NzNjYjAwNzBmOGExNjQxNDY3ODhiYzc1ZjMzYTNhOGNjNAw Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
,
Dec 20
[Sheriff]
Cannot read property 'has' of null", source: chrome://resources/cr_elements/cr_slider/cr_slider.js (290)
Cannot read property 'add' of null (chrome://resources/cr_elements/cr_slider/cr_slider.js:346)
This means both this.deltaKeyMap_ and this.draggingEventTracker_ are null, which means attached() was never called.
[9056:9056:1219/200229.695802:ERROR:CONSOLE(48)] "Mocha test failed: cr-slider key events
Error: Uncaught TypeError: Cannot read property 'has' of null (chrome://resources/cr_elements/cr_slider/cr_slider.js:290)
at keyEventOn (file:///b/s/w/ir/third_party/polymer/v1_0/components-chromium/iron-test-helpers/mock-interactions.js:398:12)
at keyDownOn (file:///b/s/w/ir/third_party/polymer/v1_0/components-chromium/iron-test-helpers/mock-interactions.js:411:5)
at Object.pressAndReleaseKeyOn (file:///b/s/w/ir/third_party/polymer/v1_0/components-chromium/iron-test-helpers/mock-interactions.js:438:5)
at pressArrowRight (file:///b/s/w/ir/chrome/test/data/webui/cr_elements/cr_slider_test.js:26:22)
at Context.test (file:///b/s/w/ir/chrome/test/data/webui/cr_elements/cr_slider_test.js:81:5)
", source: file:///b/s/w/ir/chrome/test/data/webui/mocha_adapter.js (48)
[9056:9056:1219/200229.695852:ERROR:CONSOLE(290)] "Uncaught TypeError: Cannot read property 'has' of null", source: chrome://resources/cr_elements/cr_slider/cr_slider.js (290)
[9056:9056:1219/200229.697206:ERROR:CONSOLE(48)] "Mocha test failed: cr-slider key events
AssertionError: expected 0 to equal 1
at Function.assert.strictEqual (file:///b/s/w/ir/third_party/chaijs/chai.js:2277:32)
at assertEquals (file:///b/s/w/ir/chrome/test/data/webui/test_api.js:923:15)
at Context.test (file:///b/s/w/ir/chrome/test/data/webui/cr_elements/cr_slider_test.js:82:5)
", source: file:///b/s/w/ir/chrome/test/data/webui/mocha_adapter.js (48)
[9056:9056:1219/200229.701095:ERROR:CONSOLE(48)] "Mocha test failed: cr-slider mouse events
Error: Uncaught TypeError: Cannot read property 'add' of null (chrome://resources/cr_elements/cr_slider/cr_slider.js:346)
at pointerEvent (file:///b/s/w/ir/chrome/test/data/webui/cr_elements/cr_slider_test.js:59:14)
at pointerDown (file:///b/s/w/ir/chrome/test/data/webui/cr_elements/cr_slider_test.js:67:5)
at Context.test (file:///b/s/w/ir/chrome/test/data/webui/cr_elements/cr_slider_test.js:131:5)
", source: file:///b/s/w/ir/chrome/test/data/webui/mocha_adapter.js (48)
[9056:9056:1219/200229.701134:ERROR:CONSOLE(346)] "Uncaught TypeError: Cannot read property 'add' of null", source: chrome://resources/cr_elements/cr_slider/cr_slider.js (346)
[9056:9056:1219/200229.701309:ERROR:CONSOLE(48)] "Mocha test failed: cr-slider mouse events
TypeError: Cannot read property 'listeners_' of null
at Context.test (file:///b/s/w/ir/chrome/test/data/webui/cr_elements/cr_slider_test.js:133:52)
", source: file:///b/s/w/ir/chrome/test/data/webui/mocha_adapter.js (48)
[9056:9056:1219/200229.703440:ERROR:CONSOLE(346)] "Uncaught TypeError: Cannot read property 'add' of null", source: chrome://resources/cr_elements/cr_slider/cr_slider.js (346)
[9056:9056:1219/200229.703618:ERROR:CONSOLE(48)] "Mocha test failed: cr-slider mouse events
TypeError: Cannot read property 'listeners_' of null
at Context.test (file:///b/s/w/ir/chrome/test/data/webui/cr_elements/cr_slider_test.js:133:52)
", source: file:///b/s/w/ir/chrome/test/data/webui/mocha_adapter.js (48)
[9056:9056:1219/200229.705535:ERROR:CONSOLE(48)] "Mocha test failed: cr-slider ticks and aria
Error: Uncaught TypeError: Cannot read property 'has' of null (chrome://resources/cr_elements/cr_slider/cr_slider.js:290)
,
Dec 20
,
Dec 20
,
Dec 20
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/370bd1fbfe08db1498ba360cbc4650df90d8dbfe commit 370bd1fbfe08db1498ba360cbc4650df90d8dbfe Author: Esmael El-Moslimany <aee@chromium.org> Date: Thu Dec 20 22:58:12 2018 Settings WebUI: fix flaky test, add a wait to setup before running test cases Patchset 1 is the revert. Patchset 2 is the fix. Bug: 916888 Change-Id: I16e2fc111c8ca6e3477414f4ed893bd4da0e2087 Reviewed-on: https://chromium-review.googlesource.com/c/1387386 Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by: Sam McNally <sammc@chromium.org> Commit-Queue: Esmael El-Moslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#618366} [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/chrome/browser/resources/settings/controls/settings_slider.html [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/chrome/browser/resources/settings/controls/settings_slider.js [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/chrome/browser/resources/settings/device_page/display.html [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/chrome/test/data/webui/cr_elements/cr_slider_test.js [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/chrome/test/data/webui/settings/device_page_tests.js [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/chrome/test/data/webui/settings/settings_slider_tests.js [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/ui/file_manager/audio_player/elements/control_panel.js [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/ui/file_manager/gallery/js/image_editor/image_editor_toolbar.js [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/ui/file_manager/video_player/js/media_controls.js [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/ui/file_manager/video_player/js/video_player.js [modify] https://crrev.com/370bd1fbfe08db1498ba360cbc4650df90d8dbfe/ui/webui/resources/cr_elements/cr_slider/cr_slider.js
,
Dec 20
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by bugdroid1@chromium.org
, Dec 20