Closed
Bug 1344773
Opened 7 years ago
Closed 7 years ago
Back out bug 1298950 in preparation for bug 1341458
Categories
(WebExtensions :: General, defect)
WebExtensions
General
Tracking
(firefox54 fixed, firefox55 fixed)
RESOLVED
FIXED
mozilla55
People
(Reporter: mkaply, Assigned: mkaply)
References
Details
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
mixedpuppy
:
review+
gchang
:
approval-mozilla-aurora+
|
Details |
Bug 1298950 added support for overriding homepage to chrome_url_overrides. This actually belongs in chrome_settings_overrides where it exists in Chrome. Step one is to back out that patch in preparation for the new patch.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8844050 [details] Bug 1344773 - Backout bug 1298950. https://reviewboard.mozilla.org/r/117606/#review119374
Attachment #8844050 -
Flags: review?(mixedpuppy) → review+
Pushed by mozilla@kaply.com: https://hg.mozilla.org/integration/autoland/rev/e33e742b3302 Backout bug 1298950. r=mixedpuppy
Comment 4•7 years ago
|
||
backout bugherder |
landed on m-c https://hg.mozilla.org/mozilla-central/rev/e33e742b3302
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee | ||
Comment 6•7 years ago
|
||
Comment on attachment 8844050 [details] Bug 1344773 - Backout bug 1298950. Approval Request Comment [Feature/Bug causing the regression]: This code is using an API that we are going to replace. [User impact if declined]: No user, but API available we don't want exposed. [Is this code covered by automated tests?]: Code is being removed, so there is no test. [Has the fix been verified in Nightly?]: Yes. [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?]: Removing code. [String changes made/needed]: None.
Flags: needinfo?(mozilla)
Attachment #8844050 -
Flags: approval-mozilla-aurora?
Comment 7•7 years ago
|
||
Comment on attachment 8844050 [details] Bug 1344773 - Backout bug 1298950. Replace the API we don't want to expose. Aurora54+.
Attachment #8844050 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 8•7 years ago
|
||
backout bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/18b3bedeb4be
Updated•6 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•