fast/text/line-break-ascii.html is slow on Win7 (dbg) |
||||
Issue descriptionLooking at the flakiness dashboard: https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Ftext%2Fline-break-ascii.html&testType=webkit_tests About three quarters of runs of fast/text/line-break-ascii.html flakily timeout (TIMEOUT, TIMEOUT PASS, TIMEOUT TIMEOUT PASS, or similar statuses) on WebKit Win7 (dbg). kojii: since you added this test, it probably makes sense for you to own this.
,
Oct 21 2016
,
Feb 23 2017
,
Oct 25 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2162c6289d0bbafdfeae2aa8d9e9f7ac3e250b94 commit 2162c6289d0bbafdfeae2aa8d9e9f7ac3e250b94 Author: Koji Ishii <kojii@chromium.org> Date: Wed Oct 25 04:04:52 2017 Make word-break-all-ascii.html faster The fast/css3-text/css3-word-break/word-break-all-ascii.html is slow because it forces layout to test one break opportunity, and tests 8,836 break opportunities. This patch changes it to create all test elements first, making the forced layout down to once. TBR=eae@chromium.org Bug: 658211 Change-Id: I732ba4340037a3bf99b2b979b064e1e86c5437fb Reviewed-on: https://chromium-review.googlesource.com/734882 Reviewed-by: Koji Ishii <kojii@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#511361} [modify] https://crrev.com/2162c6289d0bbafdfeae2aa8d9e9f7ac3e250b94/third_party/WebKit/LayoutTests/fast/css3-text/css3-word-break/word-break-all-ascii-expected.txt [modify] https://crrev.com/2162c6289d0bbafdfeae2aa8d9e9f7ac3e250b94/third_party/WebKit/LayoutTests/fast/css3-text/css3-word-break/word-break-all-ascii.html
,
Oct 25 2017
,
Sep 12
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e034e76a56fabeb1ee61dfdfd8e1bda668e78c6c commit e034e76a56fabeb1ee61dfdfd8e1bda668e78c6c Author: Koji Ishii <kojii@chromium.org> Date: Wed Sep 12 17:31:33 2018 Make line-break-ascii.html faster This patch makes line-break-ascii.html faster by using the shared JavaScript code with word-break-all-ascii.html, which was made faster in r511361. Bug: 658211 , 879822 Change-Id: I172aac89c4edd96ea61d2fedaff22ed9fbd0c9d9 Reviewed-on: https://chromium-review.googlesource.com/1220949 Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by: Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#590742} [modify] https://crrev.com/e034e76a56fabeb1ee61dfdfd8e1bda668e78c6c/third_party/WebKit/LayoutTests/SlowTests [modify] https://crrev.com/e034e76a56fabeb1ee61dfdfd8e1bda668e78c6c/third_party/WebKit/LayoutTests/fast/css3-text/css3-word-break/word-break-all-ascii.html [modify] https://crrev.com/e034e76a56fabeb1ee61dfdfd8e1bda668e78c6c/third_party/WebKit/LayoutTests/fast/text/line-break-ascii-expected.txt [modify] https://crrev.com/e034e76a56fabeb1ee61dfdfd8e1bda668e78c6c/third_party/WebKit/LayoutTests/fast/text/line-break-ascii.html [add] https://crrev.com/e034e76a56fabeb1ee61dfdfd8e1bda668e78c6c/third_party/WebKit/LayoutTests/fast/text/resources/line-break-test.js |
||||
►
Sign in to add a comment |
||||
Comment 1 by bugdroid1@chromium.org
, Oct 21 2016