Closed
Bug 1439493
Opened 7 years ago
Closed 6 years ago
Intermittent browser/base/content/test/general/browser_bug495058.js | about:blank/sessionrestore: browser is focused - {} == [object XULElement] - JS frame :: chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js
Categories
(Firefox :: General, defect, P5)
Firefox
General
Tracking
()
RESOLVED
INVALID
People
(Reporter: intermittent-bug-filer, Assigned: aiakab)
References
Details
(Keywords: intermittent-failure, Whiteboard: [stockwell disabled])
Attachments
(3 files)
(deleted),
patch
|
jmaher
:
review-
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
Comment 1•7 years ago
|
||
Over the last 7 days this bug has 34 failures. All of them happen on linux64-jsdcov.
Here is the most recent log example: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=164238525&lineNumber=2431
Here is a relevant snippet from that log:
[task 2018-02-25T22:37:24.314Z] 22:37:24 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug495058.js | about:blank: browser is focused - {} == [object XULElement] - JS frame :: chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js :: <TOP_LEVEL> :: line 33
[task 2018-02-25T22:37:24.314Z] 22:37:24 INFO - Stack trace:
[task 2018-02-25T22:37:24.314Z] 22:37:24 INFO - chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js:null:33
[task 2018-02-25T22:37:24.315Z] 22:37:24 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:blank: urlbar is empty - "" == "" -
[task 2018-02-25T22:37:24.315Z] 22:37:24 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:blank: placeholder text is present - "Search or enter address" == true -
[task 2018-02-25T22:37:24.477Z] 22:37:24 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:sessionrestore: uri loaded in detached tab - "about:sessionrestore" == "about:sessionrestore" -
[task 2018-02-25T22:37:24.477Z] 22:37:24 INFO - Not taking screenshot here: see the one that was previously logged
[task 2018-02-25T22:37:24.478Z] 22:37:24 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug495058.js | about:sessionrestore: browser is focused - {} == [object XULElement] - JS frame :: chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js :: <TOP_LEVEL> :: line 33
[task 2018-02-25T22:37:24.478Z] 22:37:24 INFO - Stack trace:
[task 2018-02-25T22:37:24.478Z] 22:37:24 INFO - chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js:null:33
[task 2018-02-25T22:37:24.478Z] 22:37:24 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:sessionrestore: urlbar is empty - "" == "" -
[task 2018-02-25T22:37:24.478Z] 22:37:24 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:sessionrestore: placeholder text is present - "Search or enter address" == true -
[task 2018-02-25T22:37:24.884Z] 22:37:24 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:privatebrowsing: uri loaded in detached tab - "about:privatebrowsing" == "about:privatebrowsing" -
[task 2018-02-25T22:37:24.885Z] 22:37:24 INFO - Not taking screenshot here: see the one that was previously logged
[task 2018-02-25T22:37:24.885Z] 22:37:24 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug495058.js | about:privatebrowsing: browser is focused - {} == [object XULElement] - JS frame :: chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js :: <TOP_LEVEL> :: line 33
Flags: needinfo?(past)
Updated•7 years ago
|
Whiteboard: [stockwell needswork]
Comment hidden (Intermittent Failures Robot) |
Comment 3•7 years ago
|
||
Hey Mike, since you have worked on this test in the past, maybe there is someone on your team that could look into this?
Flags: needinfo?(past) → needinfo?(mconley)
Updated•7 years ago
|
Summary: Intermittent browser/base/content/test/general/browser_bug495058.js | about:blank: browser is focused - {} == [object XULElement] - JS frame :: chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js → Intermittent browser/base/content/test/general/browser_bug495058.js | about:blank/sessionrestore: browser is focused - {} == [object XULElement] - JS frame :: chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js
Comment 4•7 years ago
|
||
keep in mind this is only failing when we collect JS coverage via devtools debugger- I suspect this is ok to disable for that config.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 7•7 years ago
|
||
There are 54 failures in the past 7 days, all seems to happen on linux64-jsdcov opt.
Recent log failure: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=167161009&lineNumber=2603
Relevant part of the log:
[task 2018-03-10T11:01:14.294Z] 11:01:14 INFO - Buffered messages finished
[task 2018-03-10T11:01:14.294Z] 11:01:14 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug495058.js | about:blank: browser is focused - {} == [object XULElement] - JS frame :: chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js :: <TOP_LEVEL> :: line 33
[task 2018-03-10T11:01:14.294Z] 11:01:14 INFO - Stack trace:
[task 2018-03-10T11:01:14.295Z] 11:01:14 INFO - chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js:null:33
[task 2018-03-10T11:01:14.295Z] 11:01:14 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:blank: urlbar is empty - "" == "" -
[task 2018-03-10T11:01:14.296Z] 11:01:14 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:blank: placeholder text is present - "Search or enter address" == true -
[task 2018-03-10T11:01:14.377Z] 11:01:14 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:sessionrestore: uri loaded in detached tab - "about:sessionrestore" == "about:sessionrestore" -
[task 2018-03-10T11:01:14.379Z] 11:01:14 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:sessionrestore: browser is focused - [object XULElement] == [object XULElement] -
[task 2018-03-10T11:01:14.379Z] 11:01:14 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:sessionrestore: urlbar is empty - "" == "" -
[task 2018-03-10T11:01:14.381Z] 11:01:14 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:sessionrestore: placeholder text is present - "Search or enter address" == true -
[task 2018-03-10T11:01:14.739Z] 11:01:14 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:privatebrowsing: uri loaded in detached tab - "about:privatebrowsing" == "about:privatebrowsing" -
[task 2018-03-10T11:01:14.740Z] 11:01:14 INFO - Not taking screenshot here: see the one that was previously logged
[task 2018-03-10T11:01:14.740Z] 11:01:14 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/general/browser_bug495058.js | about:privatebrowsing: browser is focused - {} == [object XULElement] - JS frame :: chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js :: <TOP_LEVEL> :: line 33
[task 2018-03-10T11:01:14.740Z] 11:01:14 INFO - Stack trace:
[task 2018-03-10T11:01:14.740Z] 11:01:14 INFO - chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug495058.js:null:33
[task 2018-03-10T11:01:14.741Z] 11:01:14 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:privatebrowsing: urlbar is empty - "" == "" -
[task 2018-03-10T11:01:14.742Z] 11:01:14 INFO - TEST-PASS | browser/base/content/test/general/browser_bug495058.js | about:privatebrowsing: placeholder text is present - "Search or enter address" == true -
[task 2018-03-10T11:01:14.789Z] 11:01:14 INFO - Leaving test bound
Can someone please look at this?
Flags: needinfo?(past)
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 9•7 years ago
|
||
Assignee: nobody → aiakab
Attachment #8958020 -
Flags: review?(jmaher)
Comment 10•7 years ago
|
||
Comment on attachment 8958020 [details] [diff] [review]
Disabled on Linux jsdcov platform
Review of attachment 8958020 [details] [diff] [review]:
-----------------------------------------------------------------
::: browser/base/content/test/general/browser.ini
@@ +162,4 @@
> [browser_bug491431.js]
> # DO NOT ADD MORE TESTS HERE. USE A TOPICAL DIRECTORY INSTEAD.
> [browser_bug495058.js]
> +skip-if = (os == "linux" && jsdcov) # Bug 1439493
we only run jsdcov on linux, so all that is needed is:
skip-if = jsdcov # Bug 1439493
Attachment #8958020 -
Flags: review?(jmaher) → review-
Comment 11•7 years ago
|
||
(In reply to Joel Maher ( :jmaher) (UTC-5) from comment #4)
> keep in mind this is only failing when we collect JS coverage via devtools
> debugger- I suspect this is ok to disable for that config.
Agreed, but it makes me wonder whether someone on Patrick's team may be a better fit to look at this then.
Flags: needinfo?(past) → needinfo?(pbrosset)
Comment 12•7 years ago
|
||
Hmm, I'm really not sure who would know about this. I think it makes sense to disable the test as suggested here.
I have no idea what in the debugger API does the code coverage bit. And how it could make this test fail. But maybe Jim does know.
Flags: needinfo?(pbrosset) → needinfo?(jimb)
Comment 13•7 years ago
|
||
I'm not familiar with the coverage stuff, but I ought to get familiar with it. I'll see if I can reproduce this.
Flags: needinfo?(jimb)
Assignee | ||
Comment 14•7 years ago
|
||
Updated the patch with skip-if = jsdcov
Attachment #8958820 -
Flags: review?(jmaher)
Updated•7 years ago
|
Attachment #8958820 -
Flags: review?(jmaher) → review+
Updated•7 years ago
|
Keywords: checkin-needed,
leave-open
Comment 15•7 years ago
|
||
Pushed by aiakab@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c611e124f515
disabling browser_bug495058.js on Linux jsdcov r=jmaher
Keywords: checkin-needed
Comment 16•7 years ago
|
||
bugherder |
Updated•7 years ago
|
Whiteboard: [stockwell disable-recommended] → [stockwell disabled]
Updated•7 years ago
|
Whiteboard: [stockwell disabled] → [stockwell disabled-recommended]
Assignee | ||
Comment 17•7 years ago
|
||
:jmaher This is still failing on Linux jsdcov opt. Could you please take a look if we need to modify the patch?
Flags: needinfo?(jmaher)
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 20•7 years ago
|
||
:sparky- I thought we had a way to specify |skip-if = jsdcov|, but I cannot find prior art in the source tree. Do you have more background here that I am forgetting about? Given that we skip so many tests as these run in !e10s, I am considering pushing harder to investigate JSVM data vs jsdcov and turn off jsdcov.
Flags: needinfo?(jmaher) → needinfo?(gmierz2)
Comment 21•7 years ago
|
||
It's coverage not jsdcov: https://dxr.mozilla.org/mozilla-central/search?q=mozinfo+coverage&redirect=false
We still haven't had the chance to change the jsdcov flag from 'coverage' to 'jsdcov'.
Flags: needinfo?(gmierz2)
Comment 22•7 years ago
|
||
Arthur, can you redo the patch to use |skip-if = coverage|, here is some prior art:
https://searchfox.org/mozilla-central/source/devtools/client/aboutdebugging/test/browser.ini#25
Flags: needinfo?(aiakab)
Assignee | ||
Comment 23•7 years ago
|
||
I have modified the patch.
Flags: needinfo?(aiakab)
Attachment #8961231 -
Flags: review?(jmaher)
Updated•7 years ago
|
Attachment #8961231 -
Flags: review?(jmaher) → review+
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Comment 24•7 years ago
|
||
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/64ea6814aefa
Disable browser_bug495058.js on Linux jsdcov (coverage). r=jmaher
Keywords: checkin-needed
Comment 25•7 years ago
|
||
bugherder |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•7 years ago
|
Whiteboard: [stockwell disabled-recommended] → [stockwell disabled]
Comment 27•6 years ago
|
||
The linux64-jsdcov build has been disabled, and no longer runs in taskcluster, see bug 1496791.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(mconley)
Resolution: --- → INVALID
Updated•6 years ago
|
Keywords: leave-open
Reporter | ||
Updated•5 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•