Create submenu panel-items for open tabs "more" menu
Categories
(Firefox :: Firefox View, task)
Tracking
()
Tracking | Status | |
---|---|---|
firefox119 | --- | fixed |
People
(Reporter: sclements, Assigned: sclements)
References
(Depends on 1 open bug, Blocks 4 open bugs)
Details
(Whiteboard: [fidefe-firefox-view])
Attachments
(1 file)
Breaking this out from bug 1831141, since this is more work to implement since we don't have this option with the current PanelItem and PanelList web components.
This is needed for the two items with the arrow that have the same submenus as tab context menus:
[Open tabs]
Close Tab
Move Tab >
—-------------------
Copy Link
Send Tab to Device >
The spec for the submenu's is here.
Updated•10 months ago
|
Assignee | ||
Comment 1•8 months ago
|
||
- Modify panel-list and panel-item to support submenu list items
- Add submenu items for Move Tabs and Send Tabs to Devices to open tabs
- Add test coverage for submenu items in open tabs
Pushed by sclements@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/36af6059cb2b Add submenu items to open tabs more menu r=fxview-reviewers,fluent-reviewers,desktop-theme-reviewers,sfoster,flod
Comment 3•8 months ago
|
||
Backed out changeset 36af6059cb2b (Bug 1845333) for causing failures in browser_opentabs_firefoxview_next.js CLOSED TREE
Log: https://treeherder.mozilla.org/logviewer?job_id=429812053&repo=autoland&lineNumber=16451
Backout: https://hg.mozilla.org/integration/autoland/rev/94eb416f65664a564d942491a8a252a0d05504c7
Assignee | ||
Updated•8 months ago
|
Pushed by sclements@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/424ebf6979de Add submenu items to open tabs more menu r=mstriemer,fxview-reviewers,fluent-reviewers,desktop-theme-reviewers,reusable-components-reviewers,sfoster,flod,tgiles
Comment 5•8 months ago
|
||
Backed out for causing mochitest-chrome failures on test_opentabs.html.
-
Failure log
There is also this TV job if you want to take a look before re-landing.
[task 2023-09-20T18:35:07.590Z] 18:35:07 INFO - TEST-START | browser/components/firefoxview/tests/chrome/test_opentabs.html
[task 2023-09-20T18:35:07.596Z] 18:35:07 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005 (NS_ERROR_FAILURE): file /builds/worker/checkouts/gecko/chrome/nsChromeRegistry.cpp:182
[task 2023-09-20T18:35:07.596Z] 18:35:07 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: 'NS_FAILED(rv)', file /builds/worker/checkouts/gecko/chrome/nsChromeProtocolHandler.cpp:73
[task 2023-09-20T18:35:07.645Z] 18:35:07 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: Failed to retarget HTML data delivery to the parser thread.: file /builds/worker/checkouts/gecko/parser/html/nsHtml5StreamParser.cpp:1235
[task 2023-09-20T18:35:07.710Z] 18:35:07 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: early callback, or time went backwards: '!aAllowIdleDispatch', file /builds/worker/checkouts/gecko/xpcom/threads/IdleTaskRunner.cpp:198
[task 2023-09-20T18:35:07.832Z] 18:35:07 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: early callback, or time went backwards: '!aAllowIdleDispatch', file /builds/worker/checkouts/gecko/xpcom/threads/IdleTaskRunner.cpp:198
[task 2023-09-20T18:35:07.859Z] 18:35:07 INFO - GECKO(1573) | WARNING: content window passed to PrivateBrowsingUtils.isWindowPrivate. Use isContentWindowPrivate instead (but only for frame scripts).
[task 2023-09-20T18:35:07.860Z] 18:35:07 INFO - GECKO(1573) | pbu_isWindowPrivate@resource://gre/modules/PrivateBrowsingUtils.sys.mjs:23:11
[task 2023-09-20T18:35:07.861Z] 18:35:07 INFO - GECKO(1573) | OpenTabsInView@chrome://browser/content/firefoxview/opentabs.mjs:48:54
[task 2023-09-20T18:35:07.862Z] 18:35:07 INFO - GECKO(1573) | @chrome://browser/content/firefoxview/opentabs.mjs:283:16
[task 2023-09-20T18:35:07.866Z] 18:35:07 INFO - TEST-INFO | started process screentopng
[task 2023-09-20T18:35:08.080Z] 18:35:08 INFO - TEST-INFO | screentopng: exit 0
[task 2023-09-20T18:35:08.080Z] 18:35:08 INFO - TEST-UNEXPECTED-FAIL | browser/components/firefoxview/tests/chrome/test_opentabs.html | uncaught exception - TypeError: can't access property "getSendTabTargets", this.currentWindow.gSync is undefined at connectedCallback@chrome://browser/content/firefoxview/opentabs.mjs:87:5
[task 2023-09-20T18:35:08.081Z] 18:35:08 INFO - @chrome://browser/content/firefoxview/opentabs.mjs:283:16
[task 2023-09-20T18:35:08.081Z] 18:35:08 INFO -
[task 2023-09-20T18:35:08.081Z] 18:35:08 INFO - simpletestOnerror@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:2044:18
[task 2023-09-20T18:35:08.081Z] 18:35:08 INFO - @chrome://browser/content/firefoxview/opentabs.mjs:283:16
[task 2023-09-20T18:35:08.081Z] 18:35:08 INFO - GECKO(1573) | JavaScript error: chrome://browser/content/firefoxview/opentabs.mjs, line 87: TypeError: can't access property "getSendTabTargets", this.currentWindow.gSync is undefined
[task 2023-09-20T18:35:08.081Z] 18:35:08 INFO - GECKO(1573) | MEMORY STAT | vsize 11260MB | residentFast 426MB | heapAllocated 191MB
[task 2023-09-20T18:35:08.082Z] 18:35:08 INFO - add_setup | Entering
[task 2023-09-20T18:35:08.083Z] 18:35:08 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: Please do not use mouseenter/leave events in chrome. They are slower than mouseover/out!: '!nsContentUtils::IsChromeDoc(window->GetExtantDoc())', file /builds/worker/checkouts/gecko/dom/events/EventListenerManager.cpp:436
[task 2023-09-20T18:35:08.083Z] 18:35:08 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: Please do not use mouseenter/leave events in chrome. They are slower than mouseover/out!: '!nsContentUtils::IsChromeDoc(window->GetExtantDoc())', file /builds/worker/checkouts/gecko/dom/events/EventListenerManager.cpp:436
[task 2023-09-20T18:35:08.084Z] 18:35:08 INFO - add_setup | Leaving
[task 2023-09-20T18:35:08.085Z] 18:35:08 INFO - add_task | Entering open_tab_same_window
[task 2023-09-20T18:35:08.135Z] 18:35:08 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: early callback, or time went backwards: '!aAllowIdleDispatch', file /builds/worker/checkouts/gecko/xpcom/threads/IdleTaskRunner.cpp:198
[task 2023-09-20T18:35:08.272Z] 18:35:08 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: early callback, or time went backwards: '!aAllowIdleDispatch', file /builds/worker/checkouts/gecko/xpcom/threads/IdleTaskRunner.cpp:198
[task 2023-09-20T18:35:08.343Z] 18:35:08 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: NS_ENSURE_SUCCESS(rv, NS_OK) failed with result 0x80004005 (NS_ERROR_FAILURE): file /builds/worker/checkouts/gecko/caps/BasePrincipal.cpp:1150
[task 2023-09-20T18:35:08.408Z] 18:35:08 INFO - GECKO(1573) | [Parent 1573, Main Thread] WARNING: early callback, or time went backwards: '!aAllowIdleDispatch', file /builds/worker/checkouts/gecko/xpcom/threads/IdleTaskRunner.cpp:198
[task 2023-09-20T18:35:08.418Z] 18:35:08 INFO - TEST-PASS | browser/components/firefoxview/tests/chrome/test_opentabs.html | There is one window.
Assignee | ||
Comment 6•8 months ago
|
||
Dang! I'll take a look at the test_opentabs.html
but the TV job looks like an existing intermittent with browser_recentlyclosed_firefoxview_next.js
. I could use requestLongerTimeout(N)
as a temporary measure.
Pushed by sclements@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c49ded10a66b Add submenu items to open tabs more menu r=mstriemer,fxview-reviewers,fluent-reviewers,desktop-theme-reviewers,reusable-components-reviewers,sfoster,flod,tgiles
Comment 8•8 months ago
|
||
bugherder |
Description
•