Closed Bug 1483430 Opened Last year Closed Last year

Backout fxmonitor system add-on from central and beta

Categories

(Firefox :: Firefox Monitor, defect)

62 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 63
Tracking Status
firefox62 --- fixed
firefox63 --- fixed

People

(Reporter: nhnt11, Assigned: nhnt11)

References

Details

Attachments

(2 files)

It seems that m-c is just not ready for a pure web-extension system add-on. We have the following issues which have led me to decide to give up on landing this code in m-c for now:

1. Bug 1479599 has not landed in beta yet, meaning the add-on is broken in beta even in minimal form, and shows an error in the console.
2. Combined with bug 1479599, perf regression - bug 1483172. It seems like that perf cost is one that we'll have to bear at some point or the other due to loading the extension framework early, but still...
3. Difficult-to-diagnose linux64 asan failure with WebExtension Experiment API

I was really hoping this would stick so that we could continue development within m-c, giving us access to testing infrastructure and also in the interest of not having yet-another-github-repository-for-new-features. Seems like we aren't there yet. I do hope to get this code into m-c at some point, but for now, I think we should back this out.
Comment on attachment 9000143 [details] [diff] [review]
Backout from central

Review of attachment 9000143 [details] [diff] [review]:
-----------------------------------------------------------------

Ah, that's a bummer, I'm sorry you had to go through the trouble for nothing.

I'd be interested in your alternative plan, do you intend to go through Shield? But we can talk about this on a different channel, maybe :)
Attachment #9000143 - Flags: review?(jhofmann) → review+
Attachment #9000144 - Flags: review?(jhofmann) → review+
Comment on attachment 9000144 [details] [diff] [review]
Backout from beta

Approval Request Comment
[Feature/Bug causing the regression]: bug 1480610
[User impact if declined]: Console error in beta
[Is this code covered by automated tests?]: It's a backout :)
[Has the fix been verified in Nightly?]: Not yet
[Needs manual test from QE? If yes, steps to reproduce]: Not really, but one can verify the patch by checking the error console and ensuring that it is free of firefox monitor related errors.
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: No
[Why is the change risky/not risky?]: It backs out a previous uplift
[String changes made/needed]: none
Attachment #9000144 - Flags: approval-mozilla-beta?
Comment on attachment 9000144 [details] [diff] [review]
Backout from beta

Makes sense. This should land for 62 beta 18.
Attachment #9000144 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/48bf31989df6
Backout fxmonitor system add-on from mozilla-central. r=johannh
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/48bf31989df6
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
You need to log in before you can comment on or make changes to this bug.