1722460 - browser_viewport_changed_meta.js fails with server targets

Status: RESOLVED FIXED

(DevTools :: Responsive Design Mode, defect)

Description

[Alexandre Poirot [:ochameau]]

This test fails with the following errors:

devtools/client/responsive/test/browser/browser_viewport_changed_meta.js
  FAIL Replaced meta viewport content "width=200, initial-scale=1" resolution is as expected. - Got "1.00", expected "4.00"
Stack trace:
chrome://mochikit/content/browser-test.js:test_is:1354
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/browser_viewport_changed_meta.js:null:108
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/head.js:addRDMTaskWithPreAndPost/<:192
chrome://mochikit/content/browser-test.js:Tester_execTest/<:1081
chrome://mochikit/content/browser-test.js:Tester_execTest:1121
chrome://mochikit/content/browser-test.js:nextTest/<:938
chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/<:1041
  UNEXPECTED-PASS Replaced meta viewport content "width=200, initial-scale=1" resolution matches Chrome resolution. - Got "1.00", expected "1.00"
Stack trace:
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/browser_viewport_changed_meta.js:null:115
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/head.js:addRDMTaskWithPreAndPost/<:192
chrome://mochikit/content/browser-test.js:Tester_execTest/<:1081
chrome://mochikit/content/browser-test.js:Tester_execTest:1121
chrome://mochikit/content/browser-test.js:nextTest/<:938
chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/<:1041
  FAIL Replaced meta viewport content "width=400, initial-scale=1" resolution is as expected. - Got "1.00", expected "2.00"
Stack trace:
chrome://mochikit/content/browser-test.js:test_is:1354
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/browser_viewport_changed_meta.js:null:108
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/head.js:addRDMTaskWithPreAndPost/<:192
chrome://mochikit/content/browser-test.js:Tester_execTest/<:1081
chrome://mochikit/content/browser-test.js:Tester_execTest:1121
chrome://mochikit/content/browser-test.js:nextTest/<:938
chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/<:1041
  UNEXPECTED-PASS Replaced meta viewport content "width=400, initial-scale=1" resolution matches Chrome resolution. - Got "1.00", expected "1.00"
Stack trace:
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/browser_viewport_changed_meta.js:null:115
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/head.js:addRDMTaskWithPreAndPost/<:192
chrome://mochikit/content/browser-test.js:Tester_execTest/<:1081
chrome://mochikit/content/browser-test.js:Tester_execTest:1121
chrome://mochikit/content/browser-test.js:nextTest/<:938
chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/<:1041
  FAIL Replaced meta viewport content "width=500, initial-scale=1" resolution is as expected. - Got "1.00", expected "1.60"
Stack trace:
chrome://mochikit/content/browser-test.js:test_is:1354
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/browser_viewport_changed_meta.js:null:108
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/head.js:addRDMTaskWithPreAndPost/<:192
chrome://mochikit/content/browser-test.js:Tester_execTest/<:1081
chrome://mochikit/content/browser-test.js:Tester_execTest:1121
chrome://mochikit/content/browser-test.js:nextTest/<:938
chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/<:1041
  UNEXPECTED-PASS Replaced meta viewport content "width=500, initial-scale=1" resolution matches Chrome resolution. - Got "1.00", expected "1.00"
Stack trace:
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/browser_viewport_changed_meta.js:null:115
chrome://mochitests/content/browser/devtools/client/responsive/test/browser/head.js:addRDMTaskWithPreAndPost/<:192
chrome://mochikit/content/browser-test.js:Tester_execTest/<:1081
chrome://mochikit/content/browser-test.js:Tester_execTest:1121
chrome://mochikit/content/browser-test.js:nextTest/<:938
chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/<:1041

I don't really know how initial-scale and resolution work together,
so I haven't been able to easily spot why this starts failing.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1722460 - Wait for dom-complete after reloading between each test case in browser_viewport_changed_meta.js . r=ochameau.

This makes the test to pass when server side target switching is enabled.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/468128f84c60
Wait for dom-complete after reloading between each test case in browser_viewport_changed_meta.js . r=ochameau.

Comment 3

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/468128f84c60