New issue
Advanced search Search tips

Issue 766404 link

Starred by 1 user
Status: Assigned
Owner:
dcheng@chromium.org
Cc:
mac-bugs-priority@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug



Sign in to add a comment

plugins/keyboard-events.html (webkit_layout_tests) is flaky on chromium.mac/Mac10.12 Tests

Project Member Reported by kelv...@chromium.org, Sep 19 2017 
webkit_layout_tests failing on chromium.mac/Mac10.12 Tests

Flakiness dashboard:
plugins/keyboard-events.html

Failure reason: Missing KeyUp for 'KeyC'
--- /b/s/w/iozxZzIU/layout-test-results/plugins/keyboard-events-expected.txt
+++ /b/s/w/iozxZzIU/layout-test-results/plugins/keyboard-events-actual.txt
@@ -7,6 +7,5 @@
 CONSOLE MESSAGE: Blink Test Plugin: KeyDown 'KeyB'
 CONSOLE MESSAGE: Blink Test Plugin: KeyUp 'KeyB'
 CONSOLE MESSAGE: Blink Test Plugin: KeyDown 'KeyC'
-CONSOLE MESSAGE: Blink Test Plugin: KeyUp 'KeyC'
 
 This test checks if a plugin can receive keyboard events sent from eventSender. This is a test for  Bug 34936 .

Builders failed on: 
- Mac10.12 Tests: 
  https://build.chromium.org/p/chromium.mac/builders/Mac10.12%20Tests

Comment 1 by kelv...@chromium.org, Sep 19 2017 

dcheng@, not sure if you worked in that code anymore but assigning the bug to you as you added the expectation text file originally.  Please feel free to assign to a more appropriate owner if necessary.

Comment 2 by kelv...@chromium.org, Sep 19 2017

Labels: OS-Mac

Comment 3 by kelv...@chromium.org, Sep 19 2017 

-Sheriff-Chromium as the test expectations are updated.
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 19 2017 

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

commit 9e319197e596a8b57465591c2ded1afac2ab484d
Author: kelvinp@chromium.org <kelvinp@chromium.org>
Date: Tue Sep 19 00:39:16 2017

Update expectation for flaky WebKit layout test plugins/keyboard-events.html

No-Try: true
Tbr: dcheng@chromium.org
Bug: 766404
Change-Id: Iead01f0b639a27ab11b077f7fa06dbbb2524b5df
Reviewed-on: https://chromium-review.googlesource.com/671940
Reviewed-by: Kelvin Poon <kelvinp@chromium.org>
Commit-Queue: Kelvin Poon <kelvinp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502732}
[modify] https://crrev.com/9e319197e596a8b57465591c2ded1afac2ab484d/third_party/WebKit/LayoutTests/TestExpectations

Comment 5 by timloh@chromium.org, Sep 19 2017

Labels: -Sheriff-Chromium
Removing sheriff-chromium for real.
►

Sign in to add a comment