Given the e10s issue encountered on bug 1250534 we are considering disabling the pop-out feature on the conversation window to prevent the crash scenario from happening. This bug will only be actioned if we find that bug 1250534 is not fixed on March 14th. Acceptance criteria: - Allow disablement of the pop-out button on the conversation through a preference
Bug 1250534 isn't the issue here - that's almost fixed. Its bug 1245813 that's currently the big outstanding one.
Assignee: nobody → b.mcb
Status: NEW → ASSIGNED
Note you can pop out the conversation window by dragging the title bar. I believe this patch only disables the button, do we know if it also disables the drag behavior?
Comment on attachment 8730124 [details] [review] [loop] mancas:bug1255491 > mozilla:master Hey Dave, as Ian has discovered when dragging the conversation window can be detached, so I've updated the patch to cover that case
Attachment #8730124 - Flags: review+ → review?(dcritchley)
Comment on attachment 8730124 [details] [review] [loop] mancas:bug1255491 > mozilla:master Works as expected. Tested and wasn't able to drag and button didn't show. Ship it!
Attachment #8730124 - Flags: review?(dcritchley) → review+
Manu: Per discussion in standup today, please ship this with `loop.conversationPopOut.enabled` as `true` - there's other code landing soon that should fix the e10s issues this works around, but we want this just in case we need to turn things off.
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Nice! Thanks, Manuel!
You need to log in before you can comment on or make changes to this bug.