Closed
Bug 1430018
Opened 7 years ago
Closed 7 years ago
[Wayland] - Various minor fixes
Categories
(Core :: Widget: Gtk, enhancement, P2)
Tracking
()
RESOLVED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox59 | --- | fixed |
People
(Reporter: stransky, Assigned: stransky)
References
(Blocks 1 open bug)
Details
Attachments
(8 files, 1 obsolete file)
59 bytes,
text/x-review-board-request
|
jhorak
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
jhorak
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
jhorak
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
jhorak
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
jhorak
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
jhorak
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
jhorak
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
jhorak
:
review+
|
Details |
The Wayland suport is almost functional, we're missing some minor build fixes and tweaks.
Assignee | ||
Updated•7 years ago
|
Summary: [Wayland] - Various build fixes → [Wayland] - Various minor fixes
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 11•7 years ago
|
||
mozreview-review |
Comment on attachment 8942116 [details]
Bug 1430018 - Don't use nsWindow::IsX11Display() as it's not implemented,
https://reviewboard.mozilla.org/r/212372/#review218486
Attachment #8942116 -
Flags: review?(jhorak) → review+
Comment 12•7 years ago
|
||
mozreview-review |
Comment on attachment 8942117 [details]
Bug 1430018 - Add runnable function name to NewRunnableFunction() calls,
https://reviewboard.mozilla.org/r/212374/#review218488
Attachment #8942117 -
Flags: review?(jhorak) → review+
Comment 13•7 years ago
|
||
mozreview-review |
Comment on attachment 8942119 [details]
Bug 1430018 - Don't use gdk_x11_screen_supports_net_wm_hint(_NET_WM_STATE_FULLSCREEN) on Wayland,
https://reviewboard.mozilla.org/r/212378/#review218492
::: widget/gtk/nsWindow.cpp:5088
(Diff revision 1)
> deskBounds.width, deskBounds.height,
> getter_AddRefs(screen));
> return screen.forget();
> }
>
> +#ifdef MOZ_X11
Please don't move the ifdefs there.
::: widget/gtk/nsWindow.cpp:5107
(Diff revision 1)
> nsWindow::MakeFullScreen(bool aFullScreen, nsIScreen* aTargetScreen)
> {
> LOG(("nsWindow::MakeFullScreen [%p] aFullScreen %d\n",
> (void *)this, aFullScreen));
>
> - if (!IsFullscreenSupported(mShell)) {
> +#ifdef MOZ_X11
I would not add another #ifdef there. Just make sure the mIsX11Display is set before calling IsFullscreenSupported (like proposed already).
Please add brief comment why checking for mIsX11Display there (like in Wayland fullscreen is always supported).
Giving r+ with that fixed.
Attachment #8942119 -
Flags: review?(jhorak) → review+
Comment 14•7 years ago
|
||
mozreview-review |
Comment on attachment 8942120 [details]
Bug 1430018 - Block popup input events on container widget for popups on Wayland,
https://reviewboard.mozilla.org/r/212380/#review218500
By discussion, we've agreed that this change is no longer required since eventWidget was introduced.
Attachment #8942120 -
Flags: review?(jhorak)
Comment 15•7 years ago
|
||
mozreview-review |
Comment on attachment 8942121 [details]
Bug 1430018 - Don't call gdk_flush() from expose event on Wayland as it crashes Gtk+ (Gnome BZ #773307),
https://reviewboard.mozilla.org/r/212382/#review218504
Attachment #8942121 -
Flags: review?(jhorak) → review+
Comment 16•7 years ago
|
||
mozreview-review |
Comment on attachment 8942146 [details]
Bug 1430018 - Fixed wrong profile param name at DBusRemoteClient::DoSendDBusCommandLine(),
https://reviewboard.mozilla.org/r/212408/#review218506
Attachment #8942146 -
Flags: review?(jhorak) → review+
Comment 17•7 years ago
|
||
mozreview-review |
Comment on attachment 8942147 [details]
Bug 1430018 - dlsym gdk_wayland_* routines as those are available at Gtk+ > 3.8,
https://reviewboard.mozilla.org/r/212410/#review218512
Attachment #8942147 -
Flags: review?(jhorak) → review+
Comment 18•7 years ago
|
||
mozreview-review |
Comment on attachment 8942145 [details]
Bug 1430018 - Add namespace mozilla::widget to nsNativeThemeGTK.cpp as it's needed by ScreenHelperGTK,
https://reviewboard.mozilla.org/r/212406/#review218514
Attachment #8942145 -
Flags: review?(jhorak) → review+
Comment 19•7 years ago
|
||
mozreview-review |
Comment on attachment 8942118 [details]
Bug 1430018 - Unify GtkCompositorWidgetInitData() call for X11 and Wayland,
https://reviewboard.mozilla.org/r/212376/#review218516
::: widget/gtk/nsWindow.cpp:7108
(Diff revision 1)
> - GetClientSize());
> - } else
> -#endif
> - {
> - *aInitData = mozilla::widget::GtkCompositorWidgetInitData(
> + *aInitData = mozilla::widget::GtkCompositorWidgetInitData(
> - mXWindow,
> + (mXWindow != X11None) ? mXWindow : (uintptr_t)nullptr,
Looks much better without ifdefs, could you please use
just empty nsCString() without nullptr, like:
mXDisplay ? nsCString(XDisplayString(mXDisplay)) : nsCString()
as long as I don't see nsCString(nullptr) anywhere in the sources and it looks strange to me.
r+ with that fixed.
Attachment #8942118 -
Flags: review?(jhorak) → review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8942120 -
Attachment is obsolete: true
Comment 26•7 years ago
|
||
Pushed by stransky@redhat.com:
https://hg.mozilla.org/integration/autoland/rev/509364918089
Don't use nsWindow::IsX11Display() as it's not implemented, r=jhorak
https://hg.mozilla.org/integration/autoland/rev/15f4b0d4bae5
Add runnable function name to NewRunnableFunction() calls, r=jhorak
https://hg.mozilla.org/integration/autoland/rev/4cf0168d93c5
Unify GtkCompositorWidgetInitData() call for X11 and Wayland, r=jhorak
https://hg.mozilla.org/integration/autoland/rev/7350873319fa
Don't use gdk_x11_screen_supports_net_wm_hint(_NET_WM_STATE_FULLSCREEN) on Wayland, r=jhorak
https://hg.mozilla.org/integration/autoland/rev/cb71ca2516f2
Don't call gdk_flush() from expose event on Wayland as it crashes Gtk+ (Gnome BZ #773307), r=jhorak
https://hg.mozilla.org/integration/autoland/rev/c361754494d0
Add namespace mozilla::widget to nsNativeThemeGTK.cpp as it's needed by ScreenHelperGTK, r=jhorak
https://hg.mozilla.org/integration/autoland/rev/e432cc393634
Fixed wrong profile param name at DBusRemoteClient::DoSendDBusCommandLine(), r=jhorak
https://hg.mozilla.org/integration/autoland/rev/40a2e98240cf
dlsym gdk_wayland_* routines as those are available at Gtk+ > 3.8, r=jhorak
Comment 27•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/509364918089
https://hg.mozilla.org/mozilla-central/rev/15f4b0d4bae5
https://hg.mozilla.org/mozilla-central/rev/4cf0168d93c5
https://hg.mozilla.org/mozilla-central/rev/7350873319fa
https://hg.mozilla.org/mozilla-central/rev/cb71ca2516f2
https://hg.mozilla.org/mozilla-central/rev/c361754494d0
https://hg.mozilla.org/mozilla-central/rev/e432cc393634
https://hg.mozilla.org/mozilla-central/rev/40a2e98240cf
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in
before you can comment on or make changes to this bug.
Description
•