Closed Bug 1594419 Opened 5 years ago Closed 5 years ago

Intermittent TEST-UNEXPECTED-NOTRUN | /visual-viewport/viewport-no-resize-event-on-overflow-recalc.html | Resize event not fired at window.visualViewport when content is added - expected PASS

Categories

(Core :: Layout, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: apavel)

References

(Regression)

Details

(Keywords: intermittent-failure, regression)

Attachments

(1 file)

Filed by: malexandru [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=274815376&repo=autoland
Full log: https://queue.taskcluster.net/v1/task/A2RpbfqjQyK0_n3gTPJhFA/runs/0/artifacts/public/logs/live_backing.log


[task 2019-11-06T13:27:01.981Z] 13:27:01 INFO - TEST-START | /visual-viewport/viewport-no-resize-event-on-overflow-recalc.html
[task 2019-11-06T13:27:02.001Z] 13:27:02 INFO - Setting pref dom.meta-viewport.enabled (true)
[task 2019-11-06T13:27:02.029Z] 13:27:02 INFO - Setting pref apz.allow_zooming (true)
[task 2019-11-06T13:27:02.072Z] 13:27:02 INFO - Setting pref dom.visualviewport.enabled (true)
[task 2019-11-06T13:27:42.257Z] 13:27:42 INFO -
[task 2019-11-06T13:27:42.257Z] 13:27:42 INFO - TEST-UNEXPECTED-NOTRUN | /visual-viewport/viewport-no-resize-event-on-overflow-recalc.html | Resize event not fired at window.visualViewport when content is added - expected PASS
[task 2019-11-06T13:27:42.687Z] 13:27:42 INFO - TEST-UNEXPECTED-TIMEOUT | /visual-viewport/viewport-no-resize-event-on-overflow-recalc.html | expected OK
[task 2019-11-06T13:27:42.687Z] 13:27:42 INFO - TEST-INFO took 40709ms
[task 2019-11-06T13:27:43.332Z] 13:27:43 INFO - Closing logging queue
[task 2019-11-06T13:27:43.332Z] 13:27:43 INFO - queue closed
[task 2019-11-06T13:27:43.349Z] 13:27:43 INFO - Setting up ssl
[task 2019-11-06T13:27:43.365Z] 13:27:43 INFO - certutil |
[task 2019-11-06T13:27:43.380Z] 13:27:43 INFO - certutil |
[task 2019-11-06T13:27:43.395Z] 13:27:43 INFO - certutil |
[task 2019-11-06T13:27:43.395Z] 13:27:43 INFO - Certificate Nickname Trust Attributes
[task 2019-11-06T13:27:43.395Z] 13:27:43 INFO - SSL,S/MIME,JAR/XPI
[task 2019-11-06T13:27:43.396Z] 13:27:43 INFO -
[task 2019-11-06T13:27:43.396Z] 13:27:43 INFO - web-platform-tests CT,,
[task 2019-11-06T13:27:43.396Z] 13:27:43 INFO -
[task 2019-11-06T13:27:46.309Z] 13:27:46 INFO - adb Granting important runtime permissions to org.mozilla.geckoview.test
[task 2019-11-06T13:27:47.557Z] 13:27:47 INFO - adb launch_application: am start -W -n org.mozilla.geckoview.test/org.mozilla.geckoview.test.TestRunnerActivity -a android.intent.action.MAIN --es env9 MOZ_DISABLE_NONLOCAL_CONNECTIONS=1 --es env8 R_LOG_DESTINATION=stderr --es args "-no-remote -profile /sdcard/tests/profile --marionette about:blank" --es env3 MOZ_HIDE_RESULTS_TABLE=1 --es env2 R_LOG_VERBOSE=1 --es env1 MOZ_WEBRENDER=0 --es env0 MOZ_CRASHREPORTER=1 --es env7 MOZ_CRASHREPORTER_SHUTDOWN=1 --es env6 MOZ_IN_AUTOMATION=1 --es env5 MOZ_LOG=signaling:3,mtransport:4,DataChannel:4,jsep:4 --es env4 STYLO_THREADS=4 --ez use_multiprocess True --es env12 R_LOG_LEVEL=6 --es env11 MOZ_PROCESS_LOG=/tmp/tmpzc91jTpidlog --es env10 MOZ_CRASHREPORTER_NO_REPORT=1
[task 2019-11-06T13:27:48.929Z] 13:27:48 INFO - Starting runner
[task 2019-11-06T13:27:49.076Z] 13:27:49 INFO - TEST-START | /visual-viewport/viewport-read-size-causes-layout.html

Keywords: regression

Geoff any idea why this is still failing?

previous patch was this https://phabricator.services.mozilla.com/D51808

Flags: needinfo?(gbrown)

Why is the .ini file in the tests directory? Instead of

testing/web-platform/tests/visual-viewport/viewport-no-resize-event-on-overflow-recalc.html.ini

shouldn't that be located at

testing/web-platform/meta/visual-viewport/viewport-no-resize-event-on-overflow-recalc.html.ini

?

Flags: needinfo?(gbrown)
Pushed by rgurzau@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/54819ad7decc update .ini path and viewport-no-resize-event-on-overflow-recalc.html expectations r=gbrown
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/20291 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Upstream PR was closed without merging

I think the old ini file at testing/web-platform/tests/visual-viewport/viewport-no-resize-event-on-overflow-recalc.html.ini has no effect. We only currently see failures on Android, so when we put the ini file in the correct place, it also has to be updated to only modify the expectations for Android. Something like:

[viewport-no-resize-event-on-overflow-recalc.html]
  [Resize event not fired at window.visualViewport when content is added]
    expected:
      if (os == "android"): ["PASS", "NOTRUN"]

:apavel - Can you update your patch and check it on try?

Flags: needinfo?(gbrown) → needinfo?(apavel)
Flags: needinfo?(apavel)

Revision updated.

Pushed by apavel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2d01da3fafbb update .ini path and viewport-no-resize-event-on-overflow-recalc.html expectations r=gbrown

Wrong backout.

Flags: needinfo?(gbrown)
Pushed by apavel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/84da4d581c3c update .ini path and viewport-no-resize-event-on-overflow-recalc.html expectations r=gbrown
Flags: needinfo?(gbrown)
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Assignee: nobody → apavel
Upstream PR merged by moz-wptsync-bot
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: