Chromium Code Reviews| Index: LayoutTests/fast/css/focus-ring-recursive-continuations.html |
| diff --git a/LayoutTests/fast/css/focus-ring-recursive-continuations.html b/LayoutTests/fast/css/focus-ring-recursive-continuations.html |
| index 2ece4631730c9ccb879f631a674ba35e2b6e315b..d6021ccafcc9af54db173d28857aa51618f79798 100644 |
| --- a/LayoutTests/fast/css/focus-ring-recursive-continuations.html |
| +++ b/LayoutTests/fast/css/focus-ring-recursive-continuations.html |
| @@ -4,14 +4,15 @@ |
| description('When adding focus ring rects, we should avoid adding duplicated rect for continuations'); |
| onload = function() { |
| document.body.offsetTop; |
| - // 5 focus ring rects: |
| + // 6 focus ring rects: |
| // - 0: 'SPAN0' part of the span; |
| // - 1: div DIV1 |
| - // - 2: first line box of div DIV1 |
| - // - 3: div DIV2 |
| - // - 4: first line box of div DIV2 |
| + // - 2: anonymous block containing the inline part of DIV1 before DIV2 |
| + // - 3: first line box of div DIV1 |
| + // - 4: div DIV2 |
| + // - 5: first line box of div DIV2 |
| if (window.testRunner && window.internals) |
| - shouldBe("internals.focusRingRects(document.getElementById('focus'))", "5"); |
| + shouldBe("internals.outlineRects(document.getElementById('focus')).length", "6"); |
|
chrishtr
2015/08/05 03:57:50
Why are there now 6? Was this test just busted?
Xianzhu
2015/08/05 15:58:42
Actually there is no behavior change, but the test
|
| }; |
| </script> |
| <div> |