1253688 - Loop broken in e10s (chat window doesn't open)

Status: RESOLVED FIXED

(Hello (Loop) :: Client, defect)

Description

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

STR:
1. Start a Nightly build.
2. Set the pref "loop.remote.autostart" to true.
3. Restart.
4. Click on the Hello icon (smiley face) in the toolbar.
5. Click through the intro material if you haven't already and then click again.
6. Click "Browse this page with a friend".

ER:
A chat window pops up. Right now the camera functionality seems to be broken, which I think is a separate bug. But you should at least see a chat window in the lower right corner.

AR:
Nothing pops up and the browser doesn't respond when you mouse over tabs or buttons. If I open a new tab, then the chat window appears. Seems like a drawing issue.

I bisected this regression to bug 1229429. Jim, can you take a look? If I back that patch out locally, the problem goes away.

Comment 1

[Jim Mathies [:jimm]]

That is surprising since that code only affects windowed plugin display. I'll try to reproduce.

Comment 2

[Jim Mathies [:jimm]]

Ok I can reproduce. Fixing this is easy but I'd also like to try and avoid regressing bug 1229429. Playing around with an alternative fix.

Comment 3

[Jim Mathies [:jimm]]

I think I'll just back out 1229429 so I can uplift the change to beta.

Comment 4

[Jim Mathies [:jimm]]

Attachment: backout patch

Bug 1253688 fixed a non-blocking e10s specific bug related plugins. This should be safe to take all the way up to beta 45 since 45 won't enable e10s in release. The backout fixes a really bad loop regression in a chat window loop creates when you open the "browse this page with a friend" feature.

I'm going to work on an alternative fix for bug 1253688 with the hope of getting it fixed in 47 or 48 in time for e10s rollout to release.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2523b65f4676

Comment 6

[Jim Mathies [:jimm]]

Comment on attachment 8726979 [details] [diff] [review]
backout patch

Approval Request Comment
[Feature/regressing bug #]:
e10s specific bug caused by bug 1253688
[User impact if declined]:
Bug 1253688 fixed a non-blocking e10s specific bug related plugins. This should be safe to take all the way up to beta 45 since 45 won't enable e10s in release. The backout fixes a really bad loop regression in a chat window loop creates when you open the "browse this page with a friend" feature.
[Describe test coverage new/current, TreeHerder]:
just landed on mc, trying to get into aurora before the merge. Could wait for the beta 2 build if necessary.
[Risks and why]: 
low - well understood code, e10s specific
[String/UUID change made/needed]:
none

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/2523b65f4676

Comment 8

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8726979 [details] [diff] [review]
backout patch

Fix for Loop/e10s regression, let's try to get this into aurora for the 46 beta 1 build.

Comment 9

[Mark Banner (:standard8)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/3ac92b42152f