1565401 - [wayland] Bookmarks menu is no longer scrollable

Status: RESOLVED FIXED

(Core :: Widget: Gtk, defect, P2)

Description

[Asif Youssuff]

Steps to reproduce:

1. Create enough bookmarks that it doesn't fit on a single screen (vertically in a list)
2. click the bookmarks menu (menu bar)

What happens:

The bookmarks menu is missing arrows at the bottom of the menu and is not scrollable via mousewheel; additional bookmarks are not visible. This also applies to subfolders.

Expected result:

User should be able to access all bookmarks via the menu bar.

25:22.56 INFO: No more inbound revisions, bisection finished.
25:22.56 INFO: Last good revision: e8b8985aa697ef4dad6ed72957186c658669bc1d
25:22.56 INFO: First bad revision: 7d383b757d50494cb68ba7b1129d6457765a3314
25:22.56 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=e8b8985aa697ef4dad6ed72957186c658669bc1d&tochange=7d383b757d50494cb68ba7b1129d6457765a3314

Comment 1

[Emilio Cobos Álvarez (:emilio)]

I just noticed this as well by chance, since I was asked to investigate some issues with bug 1539651.

Probably we still want to constraint the height to the screen height, or something of that sort...

Comment 2

[Asif Youssuff]

Attachment: Screenshot from 2019-08-10 01-41-52.png

This also affects menus in preferences - for example, the fonts dropdown. See screenshot.

Comment 3

[Martin Stránský [:stransky]]

Jan is going to look at it.

Comment 4

[Jan Horak [:jhorak]]

Attachment: Bug 1565401 - return valid screen size on multimonitor setup under Wayland; r=stransky

By getting the right screen size we can shrink the popup menus which overflows
the screen size under Wayland. The ScreenManager does not help us, because we
can't get absolute window position, but we can use gdk_display_get_monitor_at_window
and gdk_monitor_get_workarea to get the correct screen rectangle.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/23e9fe8a919b
return valid screen size on multimonitor setup under Wayland; r=stransky,NeilDeakin

Comment 6

[Cosmin Sabou [:CosminS]]

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&resultStatus=testfailed%2Cbusted%2Cexception&revision=23e9fe8a919bf7315734c8450913c803b45deac3&selectedJob=271519098

Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=271519098&repo=autoland

Back-out link: https://hg.mozilla.org/integration/autoland/rev/c6056ecd793e6511ab28803792bea16f107c82b1

[task 2019-10-16T15:37:20.309Z] 15:37:20 INFO - /builds/worker/fetches/sccache/sccache /builds/worker/fetches/gcc/bin/g++ -o nsWindow.o -c -I/builds/worker/workspace/build/src/obj-firefox/dist/stl_wrappers -I/builds/worker/workspace/build/src/obj-firefox/dist/system_wrappers -include /builds/worker/workspace/build/src/config/gcc_hidden.h -DNDEBUG=1 -DTRIMMED=1 -DOS_POSIX=1 -DOS_LINUX=1 -DCAIRO_GFX '-DMOZ_APP_NAME="firefox"' -DSTATIC_EXPORTABLE_JS_API -DMOZ_HAS_MOZGLUE -DMOZILLA_INTERNAL_API -DIMPL_LIBXUL -I/builds/worker/workspace/build/src/widget/gtk -I/builds/worker/workspace/build/src/obj-firefox/widget/gtk -I/builds/worker/workspace/build/src/obj-firefox/ipc/ipdl/_ipdlheaders -I/builds/worker/workspace/build/src/ipc/chromium/src -I/builds/worker/workspace/build/src/ipc/glue -I/builds/worker/workspace/build/src/layout/base -I/builds/worker/workspace/build/src/layout/generic -I/builds/worker/workspace/build/src/layout/xul -I/builds/worker/workspace/build/src/other-licenses/atk-1.0 -I/builds/worker/workspace/build/src/widget -I/builds/worker/workspace/build/src/widget/headless -I/builds/worker/workspace/build/src/widget/x11 -I/builds/worker/workspace/build/src/obj-firefox/dist/include -I/builds/worker/workspace/build/src/obj-firefox/dist/include/nspr -I/builds/worker/workspace/build/src/obj-firefox/dist/include/nss -fPIC -DMOZILLA_CLIENT -include /builds/worker/workspace/build/src/obj-firefox/mozilla-config.h -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -Wall -Wempty-body -Wignored-qualifiers -Woverloaded-virtual -Wpointer-arith -Wsign-compare -Wtype-limits -Wunreachable-code -Wwrite-strings -Wno-invalid-offsetof -Wduplicated-cond -Wno-error=maybe-uninitialized -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wno-error=coverage-mismatch -Wno-error=free-nonheap-object -Wformat -D_GLIBCXX_USE_CXX11_ABI=0 -fno-sized-deallocation -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fstack-protector-strong -fno-exceptions -fno-strict-aliasing -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -g -freorder-blocks -O2 -fno-omit-frame-pointer -funwind-tables -Werror -I/builds/worker/workspace/build/src/widget/gtk/compat-gtk3 -pthread -I/usr/include/gtk-3.0 -I/usr/include/pango-1.0 -I/usr/include/gio-unix-2.0/ -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/freetype2 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/libpng12 -I/usr/include/gtk-3.0/unix-print -I/usr/include/dbus-1.0 -I/usr/lib/x86_64-linux-gnu/dbus-1.0/include -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -Wno-error=shadow -MD -MP -MF .deps/nsWindow.o.pp /builds/worker/workspace/build/src/widget/gtk/nsWindow.cpp
[task 2019-10-16T15:37:20.309Z] 15:37:20 INFO - In file included from /builds/worker/workspace/build/src/widget/gtk/nsWindow.cpp:8:0:
[task 2019-10-16T15:37:20.309Z] 15:37:20 ERROR - /builds/worker/workspace/build/src/widget/gtk/nsWindow.h:401:20: error: 'virtual nsresult nsWindow::GetScreenRect(nsIWidget::LayoutDeviceIntRect*)' marked 'override', but does not override
[task 2019-10-16T15:37:20.309Z] 15:37:20 INFO - virtual nsresult GetScreenRect(LayoutDeviceIntRect* aRect) override;
[task 2019-10-16T15:37:20.309Z] 15:37:20 INFO - ^~~~~~~~~~~~~
[task 2019-10-16T15:37:20.309Z] 15:37:20 INFO - /builds/worker/workspace/build/src/config/rules.mk:787: recipe for target 'nsWindow.o' failed
[task 2019-10-16T15:37:20.309Z] 15:37:20 ERROR - make[4]: *** [nsWindow.o] Error 1
[task 2019-10-16T15:37:20.309Z] 15:37:20 INFO - make[4]: Leaving directory '/builds/worker/workspace/build/src/obj-firefox/widget/gtk'
[task 2019-10-16T15:37:20.309Z] 15:37:20 INFO - make[4]: *** Waiting for unfinished jobs....
[task 2019-10-16T15:37:20.310Z] 15:37:20 INFO - make[4]: Entering directory '/builds/worker/workspace/build/src/obj-firefox/widget'
[task 2019-10-16T15:37:20.310Z] 15:37:20 INFO - widget/Unified_cpp_widget0.o

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/48951378e166
return valid screen size on multimonitor setup under Wayland; r=stransky,NeilDeakin

Comment 8

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/48951378e166

Comment 9

[Laurențiu Nicola]

This seems to have regressed autoscroll on Wayland, bug 1589949.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

I assume this fix can ride the regular trains, but feel free to nominate for ESR68 approval if this is needed there.

Comment 11

[Laurențiu Nicola]

This also seems to have caused an issue with drop-down pop-ups not being drawn on Wayland when using multiple monitors: bug 1591936.