Closed Bug 1513436 Opened 7 years ago Closed 7 years ago

Intermittent testing\marionette\harness\marionette_harness\tests\unit\test_navigation.py TestBackForwardNavigation.test_frameset | UnknownException: Failed to trigger opening a new tab: TypeError: windowId is undefined

Categories

(Testing :: Marionette Client and Harness, defect, P5)

Product:
Testing
Component:
Marionette Client and Harness
Version:
Version 3
Type:
defect

Tracking

(firefox65 fixed, firefox66 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla66
Tracking Flags:
Tracking Status
firefox65 --- fixed
firefox66 --- fixed

People

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

References

Details

(Keywords: intermittent-failure)

Filed by: ccoroiu [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=216491580&repo=mozilla-beta https://queue.taskcluster.net/v1/task/OlZIHsPTTnGWAK0Z0YuVTA/runs/0/artifacts/public/logs/live_backing.log 00:08:21 INFO - GeckoDriver.prototype.newWindow@chrome://marionette/content/driver.js:2791:11 00:08:22 INFO - 1544573302807 Marionette DEBUG 495 -> [0,11,"Marionette:GetContext",{}] 00:08:22 INFO - 1544573302808 Marionette DEBUG 495 <- [1,11,null,{"value":"content"}] 00:08:22 INFO - 1544573302809 Marionette DEBUG 495 -> [0,12,"Marionette:SetContext",{"value":"chrome"}] 00:08:22 INFO - 1544573302810 Marionette DEBUG 495 <- [1,12,null,{"value":null}] 00:08:22 INFO - 1544573302811 Marionette DEBUG 495 -> [0,13,"WebDriver:TakeScreenshot",{"highlights":null,"full":true,"hash":false,"id":null,"scroll":true}] 00:08:22 INFO - [Parent 1152, Main Thread] WARNING: Remote iframe not rendered: file z:/build/build/src/layout/ipc/RenderFrame.cpp, line 271 00:08:22 INFO - 1544573302855 Marionette DEBUG 495 <- [1,13,null,{"value":"iVBORw0KGgoAAAANSUhEUgAABQAAAAQuCAYAAACDNZAFAAAgAElEQVR4nOzdf5RU9Z3g/UqeZ87k2cnZZ/YZ9+xkMzsmOGPGZKKTiUlM ... ACAAAAwJgABAAAAIAxAQgAAAAAYwIQAAAAAMYEIAAAAACMCUAAAAAAGBOAAAAAADAmAAEAAABgTAACAAAAwJgABAAAAICxAMnqYGgwhfqgAAAAAElFTkSuQmCC"}] 00:08:22 INFO - 1544573302857 Marionette DEBUG 495 -> [0,14,"Marionette:SetContext",{"value":"content"}] 00:08:22 INFO - 1544573302858 Marionette DEBUG 495 <- [1,14,null,{"value":null}] 00:08:22 INFO - 1544573302859 Marionette DEBUG 495 -> [0,15,"Marionette:GetContext",{}] 00:08:22 INFO - 1544573302859 Marionette DEBUG 495 <- [1,15,null,{"value":"content"}] 00:08:22 INFO - 1544573302860 Marionette DEBUG 495 -> [0,16,"Marionette:SetContext",{"value":"content"}] 00:08:22 INFO - 1544573302861 Marionette DEBUG 495 <- [1,16,null,{"value":null}] 00:08:22 INFO - 1544573302862 Marionette DEBUG 495 -> [0,17,"WebDriver:GetPageSource",{}] 00:08:23 INFO - --DOMWINDOW == 11 (0F222400) [pid = 4948] [serial = 168] [outer = 00000000] [url = data:application/xhtml+xml,%3C%21DOCTYPE%20html%20PUBLIC%20%22-//W3C//DTD%20XHTML%201.0%20Strict//EN%22%0A%20%20%20%20%22http%3A//www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd%22%3E%0A%3Chtml%20xmlns%3D%22http%3A//www.w3.org/1999/xhtml%22%20xml%3Alang%3D%22en%22%20lang%3D%22en%22%3E%0A%20%20%3Chead%3E%0A%20%20%20%20%3Ctitle%3EXHTML%20might%20be%20the%20future%3C/title%3E%0A%20%20%3C/head%3E%0A%0A%20%20%3Cbody%3E%0A%20%20%20%20%3Cdiv%3E%3Cp%3Efoo%3C/p%3E%20%3Cp%3Ebar%3C/p%3E%3C/div%3E%0A%20%20%3C/body%3E%0A%3C/html%3E] 00:08:24 INFO - --DOMWINDOW == 80 (18307400) [pid = 1152] [serial = 115] [outer = 00000000] [url = about:blank] 00:08:24 INFO - --DOMWINDOW == 79 (18F83400) [pid = 1152] [serial = 107] [outer = 00000000] [url = about:blank] 00:08:24 INFO - --DOMWINDOW == 78 (18F0A400) [pid = 1152] [serial = 99] [outer = 00000000] [url = about:blank] 00:08:24 INFO - 1544573304752 Marionette TRACE [34359738369] Frame script registered 00:08:24 INFO - 1544573304737 Marionette DEBUG 495 <- [1,17,null,{"value":"<html><head>\n<title>Marionette Test</title>\n</head>\n<body>\n <h1 id=\"testh1\">Test Page</h1>\n <sc ... <input id=\"createDivButton\" type=\"button\" value=\"create a div\" onclick=\"addDelayedElement()\">\n\n \n</body></html>"}] 00:08:24 INFO - [Child 3996, Main Thread] WARNING: 'NS_FAILED(GetAccentColor(unused))', file z:/build/build/src/widget/windows/nsLookAndFeel.cpp, line 480 00:08:24 INFO - 1544573304760 Marionette DEBUG 495 -> [0,18,"Marionette:SetContext",{"value":"content"}] 00:08:24 INFO - 1544573304761 Marionette DEBUG 495 <- [1,18,null,{"value":null}] 00:08:24 INFO - ++DOMWINDOW == 3 (083DA800) [pid = 3996] [serial = 3] [outer = 01472300] 00:08:24 INFO - [Child 3996, Main Thread] WARNING: 'NS_FAILED(rv)', file z:/build/build/src/dom/base/nsContentUtils.cpp, line 8166 00:08:24 ERROR - TEST-UNEXPECTED-ERROR | testing\marionette\harness\marionette_harness\tests\unit\test_navigation.py TestBackForwardNavigation.test_frameset | UnknownException: Failed to trigger opening a new tab: TypeError: windowId is undefined 00:08:24 INFO - stacktrace: 00:08:24 INFO - GeckoDriver.prototype.newWindow@chrome://marionette/content/driver.js:2791:11 00:08:24 INFO - Traceback (most recent call last): 00:08:24 INFO - File "Z:\task_1544571112\build\venv\lib\site-packages\marionette_harness\marionette_test\testcases.py", line 140, in run 00:08:24 INFO - self.setUp() 00:08:24 INFO - File "Z:\task_1544571112\build\tests\marionette\tests\testing\marionette\harness\marionette_harness\tests\unit\test_navigation.py", line 59, in setUp 00:08:24 INFO - self.new_tab = self.open_tab() 00:08:24 INFO - File "Z:\task_1544571112\build\venv\lib\site-packages\marionette_harness\runner\mixins\window_manager.py", line 75, in open_tab 00:08:24 INFO - reraise(exc, 'Failed to trigger opening a new tab: {}'.format(val), tb) 00:08:24 INFO - File "Z:\task_1544571112\build\venv\lib\site-packages\marionette_harness\runner\mixins\window_manager.py", line 68, in open_tab 00:08:24 INFO - result = self.marionette.open(type="tab", focus=focus) 00:08:24 INFO - File "Z:\task_1544571112\build\venv\lib\site-packages\marionette_driver\marionette.py", line 1443, in open 00:08:24 INFO - return self._send_message("WebDriver:NewWindow", body) 00:08:24 INFO - File "Z:\task_1544571112\build\venv\lib\site-packages\marionette_driver\decorators.py", line 26, in _ 00:08:24 INFO - return func(*args, **kwargs) 00:08:24 INFO - File "Z:\task_1544571112\build\venv\lib\site-packages\marionette_driver\marionette.py", line 764, in _send_message 00:08:24 INFO - self._handle_error(err) 00:08:24 INFO - File "Z:\task_1544571112\build\venv\lib\site-packages\marionette_driver\marionette.py", line 784, in _handle_error 00:08:24 INFO - raise errors.lookup(error)(message, stacktrace=stacktrace) 00:08:24 INFO - TEST-INFO took 5496ms
A new failure in the code as landed for bug 1504756. 00:08:19 INFO - 1544573299308 Marionette DEBUG 495 -> [0,10,"WebDriver:NewWindow",{"type":"tab","focus":false}] 00:08:19 INFO - 1544573299318 Marionette TRACE Received DOM event TabOpen for [object XULElement] [..] 00:08:19 INFO - 1544573299776 Marionette TRACE [34359738369] Frame script loaded 00:08:21 INFO - 1544573301405 Marionette DEBUG 495 <- [1,10,{"error":"unknown error","message":"TypeError: windowId is undefined","stacktrace":"GeckoDriver.prototype.newWindow@chrome://marionette/content/driver.js:2791:11\n"},null] 00:08:21 INFO - Marionette threw an error: TypeError: windowId is undefined 00:08:21 INFO - GeckoDriver.prototype.newWindow@chrome://marionette/content/driver.js:2791:11 The problem is here: https://searchfox.org/mozilla-central/rev/2e5e28f518524f8af5b158ddb605022b6a2d68cf/testing/marionette/driver.js#2791 I don't understand how this can fail given that the poll promise above only returns when there is a valid outer window id, which is also contained in the list of current internal window handles.
Blocks: 1504756
status-firefox65: --- → affected
status-firefox66: --- → affected
See Also: → 1514989
See Also: → 1514706
Blocks: 1512871
Blocks: 1514989, 1514706
See Also: 1514989, 1514706 →
The patch from bug 1504756 got backed out. As such this failure is no longer existent.
Assignee: nobody → hskupin
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox65: affected → fixed
status-firefox66: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Product: Testing → Remote Protocol
Moving bug to Testing::Marionette Client and Harness component per bug 1815831.
Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in before you can comment on or make changes to this bug.