Closed Bug 1424974 Opened 3 years ago Closed 3 years ago

[e10s] fix regression at Bug 1423810

Categories

(Core :: Widget: Gtk, enhancement)

x86_64
Linux
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: stransky, Assigned: stransky)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

+++ This bug was initially created as a clone of Bug #1423810 +++

We need to provide better fix for Bug 1423810 and enable titlebar rendering in CSD mode.
It's reproducible with e10s enabled only.
Summary: fix regression at Bug 1423810 and enable titlebar rendering in CSD mode → [e10s] fix regression at Bug 1423810 and enable titlebar rendering in CSD mode
Looks related to Bug 1196494 - property_notify_event_cb() is not called for mShell when it has child mContainer window.
Attached patch patchSplinter Review
WIP but functional patch, storing here for further work.
Summary: [e10s] fix regression at Bug 1423810 and enable titlebar rendering in CSD mode → [e10s] fix regression at Bug 1423810
Comment on attachment 8937943 [details]
Bug 1424974 - Honor mShell as top-level window,

https://reviewboard.mozilla.org/r/208642/#review214794

::: widget/gtk/nsWindow.cpp:3897
(Diff revision 1)
>          break;
>      }
>  
>      // label the drawing window with this object so we can find our way home
>      g_object_set_data(G_OBJECT(mGdkWindow), "nsWindow", this);
> +    if (drawToContainer) {

Please add brief description why do you need to add nsWindow reference to the mShell.
Attachment #8937943 - Flags: review?(jhorak) → review+
Pushed by stransky@redhat.com:
https://hg.mozilla.org/integration/autoland/rev/872897434908
Honor mShell as top-level window, r=jhorak
https://hg.mozilla.org/mozilla-central/rev/872897434908
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
See Also: → 1441665
You need to log in before you can comment on or make changes to this bug.