Bug 196298 - [iOS WK2] fast/viewport/ios/use-minimum-device-width-for-page-without-viewport-meta.html is a flaky failure
Summary: [iOS WK2] fast/viewport/ios/use-minimum-device-width-for-page-without-viewpor...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Wenson Hsieh
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-03-27 08:45 PDT by Shawn Roberts
Modified: 2019-10-25 19:36 PDT (History)
11 users (show)

See Also:

Attachments
Patch (7.89 KB, patch)
2019-10-25 14:17 PDT, Wenson Hsieh 		no flags Details | Formatted Diff | Diff
Fix a ChangeLog typo (7.88 KB, patch)
2019-10-25 14:20 PDT, Wenson Hsieh 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Shawn Roberts 2019-03-27 08:45:36 PDT 
The following layout test is failing on iOS Simulator Release WK2

fast/viewport/ios/use-minimum-device-width-for-page-without-viewport-meta.html

Probable cause:

Test has been a flaky failure since the test was added in https://trac.webkit.org/changeset/238946/webkit . Failing around 4 times on average in 500 iterations from r238946 up until r240911

Changes made in https://trac.webkit.org/changeset/240912/webkit made test fail a lot more frequently. During testing found from 240912 up until current r243537 is failing approximately 10 times 60 attempts (setting testing to exit after 10 failures)

Reproduced with : 

run-webkit-tests fast/viewport/ios/use-minimum-device-width-for-page-without-viewport-meta.html --iterations 500 -f --ios-simulator --exit-after-n-failures=10

Flakiness Dashboard:

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fviewport%2Fios%2Fuse-minimum-device-width-for-page-without-viewport-meta.html

--- /Volumes/Data/slave/ios-simulator-12-release-tests-wk2/build/layout-test-results/fast/viewport/ios/use-minimum-device-width-for-page-without-viewport-meta-expected.txt
+++ /Volumes/Data/slave/ios-simulator-12-release-tests-wk2/build/layout-test-results/fast/viewport/ios/use-minimum-device-width-for-page-without-viewport-meta-actual.txt
@@ -1,7 +1,7 @@
 setMinimumEffectiveWidth(640.00)
-window size: [640, 1096]
+window size: [320, 548]
 square size: [64, 110]
-zoom scale: 0.50
+zoom scale: 1.00
 
 setMinimumEffectiveWidth(768.00)
 window size: [768, 1315]
Comment 1 Shawn Roberts 2019-03-27 08:46:47 PDT 
Adding Yongjun Zhang who created the test.

Adding Simon Fraser who authored the change that made test more flaky.
Comment 2 Radar WebKit Bug Importer 2019-03-27 08:48:38 PDT 
<rdar://problem/49337021>
Comment 3 Shawn Roberts 2019-03-27 09:45:12 PDT 
Marked flaky in https://trac.webkit.org/changeset/243548/webkit
Comment 4 Wenson Hsieh 2019-10-25 14:17:43 PDT Comment hidden (obsolete)
Comment 5 Wenson Hsieh 2019-10-25 14:20:41 PDT 
Created attachment 381966 [details]
Fix a ChangeLog typo
Comment 6 WebKit Commit Bot 2019-10-25 19:36:27 PDT 
Comment on attachment 381966 [details]
Fix a ChangeLog typo

Clearing flags on attachment: 381966

Committed r251625: <https://trac.webkit.org/changeset/251625>
Comment 7 WebKit Commit Bot 2019-10-25 19:36:29 PDT 
All reviewed patches have been landed.  Closing bug.