middleClickAutoscroll-latching.html doesn't call second test |
|||
Issue descriptionThe middleClickAutoscroll-latching.html layout test appears to have a bug. There are two tests contained within the html, "testBubble" and "testIFrame", and the code in testBubble appears to try to call testIFrame after it succeeds. But that call happens after testBubble.done(), which causes testIFrame() to not get hit.
,
Sep 24
CCing jdragon.bae@gmail.com and adding myself as the owner for now.
,
Oct 9
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8dcc25c6ba139f655078df8c28df8e4c2395f380 commit 8dcc25c6ba139f655078df8c28df8e4c2395f380 Author: Jaeyong Bae <jdragon.bae@gmail.com> Date: Tue Oct 09 15:02:27 2018 Fix execution flow in middleClickAutoscroll test This patch means change flow of testIFrame test. Bug: 888571 Change-Id: Iec7ed393a56a407a6715fe2baf1f4cc0b067b1bc Reviewed-on: https://chromium-review.googlesource.com/c/1241114 Commit-Queue: Jaeyong Bae <jdragon.bae@gmail.com> Reviewed-by: David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#597916} [modify] https://crrev.com/8dcc25c6ba139f655078df8c28df8e4c2395f380/third_party/WebKit/LayoutTests/TestExpectations [modify] https://crrev.com/8dcc25c6ba139f655078df8c28df8e4c2395f380/third_party/WebKit/LayoutTests/fast/events/middleClickAutoscroll-latching.html
,
Oct 9
Nice work jdragon.bae@ |
|||
►
Sign in to add a comment |
|||
Comment 1 by jdragon....@gmail.com
, Sep 24