1340195 - Remove e10s block for LastPass

Status: RESOLVED FIXED

(Firefox :: Extension Compatibility, defect, P1)

Description

[:Felipe Gomes (needinfo for replies!)]

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

The e10s block for LastPass added in bug 1332692 can be removed.

We can't just back out 1332692 because we need to increase the add-on version instead of rolling it back.

Comment 1

[:Felipe Gomes (needinfo for replies!)]

[Tracking Requested - why for this release]:

Revert e10s LastPass block landed for 52/esr

Comment 2

[:Felipe Gomes (needinfo for replies!)]

Attachment: Bug 1340195 - Remove e10s block for LastPass.

Review commit: https://reviewboard.mozilla.org/r/113102/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/113102/

Comment 3

[:Felipe Gomes (needinfo for replies!)]

I'll wait until the e10s meeting tomorrow to double check on this before going forward

Comment 4

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 8838113 [details]
Bug 1340195 - Remove e10s block for LastPass.

https://reviewboard.mozilla.org/r/113102/#review114958

Per meeting today - I think we only want to do this for Beta.

Comment 5

[Julien Cristau [:jcristau]]

Is there any more action needed for 52 here, or are we keeping the lastpass block after all?

Comment 6

[:Felipe Gomes (needinfo for replies!)]

Ok, so the plan from the last e10s meeting was: unblock in beta, keep blocked in release.  Instead of writing yet another patch to do this conditionally on the update channel (and dealing with approvals and late uplifts), I plan to just let the block ride to release and then remove it only from beta after the merge (or right now in aurora)

Comment 7

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8838113 [details]
Bug 1340195 - Remove e10s block for LastPass.

Hey Mike, re-requesting review here. Instead of modifying this patch to only apply to beta and not release, I let the previous block ride to release, and then I plan to only land this on central and aurora this week so that the block is removed for the next beta.

Comment 8

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 8838113 [details]
Bug 1340195 - Remove e10s block for LastPass.

https://reviewboard.mozilla.org/r/113102/#review118338

felipe says that the plan of record is to only have the block exist on the Release channel for a single release (52). This block-removal will land on central and aurora, and will merge in the next uplift to beta (so it should be in beta 53).

The hope is that their issues will be sorted by the time 53 goes out the door. If not, we'll put the block back in 53.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4ff72c2c865f
Remove e10s block for LastPass. r=mconley

Comment 10

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8838113 [details]
Bug 1340195 - Remove e10s block for LastPass.

Approval Request Comment
[Feature/Bug causing the regression]: remove the block of e10s for LastPass v4 users for the next beta
[User impact if declined]: people using the beta versions of LastPass (v4) will continue to be blocked from e10s
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: addons blocking only applies to beta/release
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]: LastPass v4 is only on beta stage and is not the direct download found on AMO (you need to navigate to "Other versions" to find it). The currently published v3 is blocked unconditionally by being mpc=false
[String changes made/needed]: none

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/4ff72c2c865f

Comment 12

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

Comment on attachment 8838113 [details]
Bug 1340195 - Remove e10s block for LastPass.

Taking off the block for 53 here. We need this to land before the patches from bug 1342490

Comment 13

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/b0097ac9fa56