Bug 215772 – REGRESSION (r266028): platform/ios/ios/fast/coordinates/range-client-rects.html

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Aakash Jain]

platform/ios/ios/fast/coordinates/range-client-rects.html seems to be broken on iOS since r266028.

History: https://results.webkit.org/?suite=layout-tests&test=platform%2Fios%2Fios%2Ffast%2Fcoordinates%2Frange-client-rects.html


e.g.: https://ews-build.s3-us-west-2.amazonaws.com/iOS-13-Simulator-WK2-Tests-EWS/r407098-24207-clean-tree/platform/ios/ios/fast/coordinates/range-client-rects-diff.txt
-PASS range.getClientRects().length is 2
+FAIL range.getClientRects().length should be 2. Was 1.
 unscaled
 PASS clientRect.left is 0
 PASS clientRect.right is 1500
@@ -8,18 +8,18 @@
 PASS clientRect.bottom is 1500
 PASS clientRect.width is 1500
 PASS clientRect.height is 1500
-PASS clientRect.left is 0
-PASS clientRect.right is 1500
-PASS clientRect.top is 0
-PASS clientRect.bottom is 1500
-PASS clientRect.width is 1500
-PASS clientRect.height is 1500
-PASS clientRect.left is 100
-PASS clientRect.right is 200
-PASS clientRect.top is 100
-PASS clientRect.bottom is 200
-PASS clientRect.width is 100
-PASS clientRect.height is 100
+FAIL clientRect.left should be 0. Threw exception TypeError: undefined is not an object (evaluating 'clientRect.left')
+FAIL clientRect.right should be 1500. Threw exception TypeError: undefined is not an object (evaluating 'clientRect.right')
+FAIL clientRect.top should be 0. Threw exception TypeError: undefined is not an object (evaluating 'clientRect.top')
+FAIL clientRect.bottom should be 1500. Threw exception TypeError: undefined is not an object (evaluating 'clientRect.bottom')
+FAIL clientRect.width should be 1500. Threw exception TypeError: undefined is not an object (evaluating 'clientRect.width')
+FAIL clientRect.height should be 1500. Threw exception TypeError: undefined is not an object (evaluating 'clientRect.height')
+FAIL clientRect.left should be 100. Was 0.
+FAIL clientRect.right should be 200. Was 1500.
+FAIL clientRect.top should be 100. Was 0.
+FAIL clientRect.bottom should be 200. Was 1500.
+FAIL clientRect.width should be 100. Was 1500.
+FAIL clientRect.height should be 100. Was 1500.

[Hector Lopez]

Test expectation while investigated:

https://trac.webkit.org/changeset/266073/webkit

[Darin Adler]

Strange; I saw this with an earlier version of my patch, but I thought I had fixed it. Note that we don’t see the failure in EWS.

[Darin Adler]

Any clue why we didn’t see the failure in EWS?

[Aakash Jain]

(In reply to Darin Adler from comment #3)
> Any clue why we didn’t see the failure in EWS?
Yeah, I tried to explain in https://bugs.webkit.org/show_bug.cgi?id=215677#c11
Basically ios-wk2 EWS was broken for some time late last week, showing green bubbles while the tests weren't even run (due to Bug 215742). I am going to fix EWS, so that in such infrastructure failures it doesn't show green bubbles, and instead keep re-trying.

[Darin Adler]

I”ll work on fixing this.

As long as no one merges this to the branch, we have some time to get it fixed.

[Radar WebKit Bug Importer]

<rdar://problem/67708183>

[Darin Adler]

This test is peculiar. The whole thing runs just fine cross-platform. It's just that the meta tag for zooming has no effect on platforms like Mac. But the bug/regression reproduces everywhere and has nothing to do with the zooming.

I think we should consider moving it out of the platform/ios/ios directory into a cross-platform directory.

[Alexey Proskuryakov]

I tried to follow the history to see why it was iOS-only, but had to stop at r167371 (moving from Internal).

[Darin Adler]

I think the intent was to check the meta tag behavior and there just happens to be no similar bug elsewhere.

[Darin Adler]

Created attachment 407158 [details]
Patch

[Darin Adler]

Created attachment 407159 [details]
Patch

[Darin Adler]

Created attachment 407164 [details]
Patch

[Darin Adler]

Ready for review now.

[EWS]

Committed r266123: <https://trac.webkit.org/changeset/266123>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 407164 [details].