Closed
Bug 1482004
Opened 7 years ago
Closed 6 years ago
Intermittent browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Panel has not shrunk from original size (X >= Y) -
Categories
(WebExtensions :: Untriaged, defect, P5)
WebExtensions
Untriaged
Tracking
(firefox-esr60 unaffected, firefox65 wontfix, firefox66 fixed, firefox67 fixed)
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox65 | --- | wontfix |
firefox66 | --- | fixed |
firefox67 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: rpl)
References
Details
(Keywords: intermittent-failure, Whiteboard: [retriggered][stockwell fixed:1483701])
Attachments
(1 file, 3 obsolete files)
971 bytes,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
Filed by: dvarga [at] mozilla.com
https://treeherder.mozilla.org/logviewer.html#?job_id=192911950&repo=autoland
https://queue.taskcluster.net/v1/task/BUKK6neGQHOWDppv5_L1DA/runs/0/artifacts/public/logs/live_backing.log
20:07:59 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | 61 -
20:07:59 INFO - Buffered messages finished
20:07:59 INFO - TEST-UNEXPECTED-FAIL | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Panel has not shrunk from original size (672 >= 744) -
20:07:59 INFO - Stack trace:
20:07:59 INFO - chrome://mochitests/content/browser/browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js:checkPanelPosition:172
20:07:59 INFO - chrome://mochitests/content/browser/browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js:testPopupSize:222
20:07:59 INFO - chrome://mochitests/content/browser/browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js:testBrowserActionMenuResizeQuirks:283
20:07:59 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1103
20:07:59 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1094
20:07:59 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:996
20:07:59 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:795
20:07:59 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Bottom of popup should be on-screen. (695 <= 1120) -
20:07:59 INFO - Increase body children's width and height. Expect them to wrap, and the frame to grow vertically rather than widen.
20:07:59 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Browser height should increase (632 > 176) -
20:07:59 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Window width should not change -
20:07:59 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Window height should increase (632 >= 560) -
20:07:59 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Document should not be vertically scrollable - 0 <= 1 -
20:07:59 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Panel arrow is positioned as expected -
20:07:59 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | 61 -
20:07:59 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Panel has not shrunk from original size (744 >= 744) -
Comment hidden (Intermittent Failures Robot) |
Comment hidden (obsolete) |
Comment hidden (obsolete) |
Comment 4•7 years ago
|
||
Please ignore this, the comments are not related to this issue. Thank you, and my apologies.
Flags: needinfo?(mikokm)
Flags: needinfo?(ddurst)
Comment 5•7 years ago
|
||
This started to occur more often:
https://treeherder.mozilla.org/#/jobs?repo=autoland&fromchange=de3ad2046aa56e664cfd4460541d15fa81945169&filter-searchStr=OS%20X%2010.10%20debug%20Mochitests%20with%20e10s%20test-macosx64%2Fdebug-mochitest-browser-chrome-e10s-7%20M-e10s(bc7)
ddurst, can you please take a look?
Flags: needinfo?(ddurst)
Updated•7 years ago
|
Summary: Intermittent browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Panel has not shrunk from original size (672 >= 744) - → Intermittent browser/components/extensions/test/browser/browser_ext_browserAction_popup_resize.js | Panel has not shrunk from original size (X >= Y) -
Comment hidden (Intermittent Failures Robot) |
Updated•7 years ago
|
Whiteboard: [retriggered]
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 11•7 years ago
|
||
Assignee: nobody → ccoroiu
Attachment #9007418 -
Flags: review?(jmaher)
Comment 12•7 years ago
|
||
Comment on attachment 9007418 [details] [diff] [review]
Disabling browser_ext_browserAction_popup_resize.js on mac debug
Review of attachment 9007418 [details] [diff] [review]:
-----------------------------------------------------------------
::: browser/components/extensions/test/browser/browser-common.ini
@@ +59,4 @@
> [browser_ext_browserAction_popup_preload.js]
> skip-if = (os == 'win' && !debug) || (verify && debug && (os == 'mac')) # bug 1352668
> [browser_ext_browserAction_popup_resize.js]
> +skip-if = (os == 'mac' && debug) # Bug 1482004
I would really like to keep context that this fails in verify mode. Can you add in the comment that it also fails specifically in test-verify mode:
# Bug 1482004, also fails in test-verify
Attachment #9007418 -
Flags: review?(jmaher) → review-
Comment 13•7 years ago
|
||
Attachment #9007418 -
Attachment is obsolete: true
Attachment #9007425 -
Flags: review?(jmaher)
Comment 14•7 years ago
|
||
Attachment #9007425 -
Attachment is obsolete: true
Attachment #9007425 -
Flags: review?(jmaher)
Attachment #9007426 -
Flags: review?(jmaher)
Updated•7 years ago
|
Attachment #9007426 -
Flags: review?(jmaher) → review+
Updated•7 years ago
|
Keywords: checkin-needed,
leave-open
Whiteboard: [retriggered][stockwell disable-recommended] → [retriggered][stockwell disabled]
Comment 15•7 years ago
|
||
Sorry, when trying to land I've noticed that in the commit message it was another file.
Attachment #9007426 -
Attachment is obsolete: true
Attachment #9007427 -
Flags: review?(jmaher)
Comment 16•7 years ago
|
||
Comment on attachment 9007427 [details] [diff] [review]
Disabling browser_ext_browserAction_popup_resize.js on mac debugbug1482004.patch
Review of attachment 9007427 [details] [diff] [review]:
-----------------------------------------------------------------
good catch
Attachment #9007427 -
Flags: review?(jmaher) → review+
Comment 17•7 years ago
|
||
Pushed by ccoroiu@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c27c3aaafcb6
Disable browser_ext_browserAction_popup_resize.js for frequent failures r=jmaher
Keywords: checkin-needed
Comment 18•7 years ago
|
||
bugherder |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 25•6 years ago
|
||
Marking as fixed, this test has been re-enabled in Bug 1483701.
Assignee: ccoroiu → lgreco
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(ddurst)
Keywords: leave-open
Resolution: --- → FIXED
Whiteboard: [retriggered][stockwell disabled] → [retriggered][stockwell fixed:1483701]
Updated•6 years ago
|
status-firefox65:
--- → wontfix
status-firefox66:
--- → fixed
status-firefox67:
--- → fixed
status-firefox-esr60:
--- → unaffected
Target Milestone: --- → mozilla67
Comment 26•6 years ago
|
||
From my understanding this issue is related to the addon tests. Is there any need of manual QA here? If not can you please mark it as "qe-verify- "
Flags: needinfo?(lgreco)
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(lgreco) → qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•