Remove the Flyweb source from the tree

RESOLVED FIXED in Firefox 59

Status

RESOLVED FIXED
a year ago
3 months ago

People

(Reporter: jwatt, Assigned: johannh)

Tracking

Trunk
mozilla59

Firefox Tracking Flags

(firefox59 fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

a year ago
According to Kannan Flyweb is likely to be removed. Filing this bug to cover doing that.
The front-end code was implemented entirely as a System Add-on, so it can be removed very easily by getting rid of this line:

http://searchfox.org/mozilla-central/rev/1a4a26905f923458679a59a4be1e455ebc53c333/browser/extensions/moz.build#24

And then wiping out the flyweb subdirectory under browser/extensions.
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 6

a year ago
Kannan, I removed the bare minimum of code that I could find is directly related to FlyWeb. It's very possible that I forgot to remove some interfaces that were only used by FlyWeb. If you can think of any I'd appreciate pointers.
(Assignee)

Updated

a year ago
Assignee: nobody → jhofmann
Status: NEW → ASSIGNED
(Assignee)

Comment 7

a year ago
Does this need an intent to unship?

Comment 8

a year ago
mozreview-review
Comment on attachment 8931105 [details]
Bug 1374574 - Remove the FlyWeb system add-on from Firefox.

https://reviewboard.mozilla.org/r/202186/#review207546

Thanks!
Attachment #8931105 - Flags: review?(mconley) → review+

Comment 9

a year ago
mozreview-review
Comment on attachment 8931107 [details]
Bug 1374574 - Remove the FlyWeb service and DOM interfaces.

https://reviewboard.mozilla.org/r/202190/#review207676
Attachment #8931107 - Flags: review?(bzbarsky) → review+

Comment 10

a year ago
mozreview-review
Comment on attachment 8931107 [details]
Bug 1374574 - Remove the FlyWeb service and DOM interfaces.

https://reviewboard.mozilla.org/r/202190/#review207890

Thank you for taking care of this.  Do you drink beer?  Are you coming to Austin?  Count me as owing you one beverage of your choice :)
Attachment #8931107 - Flags: review?(kvijayan) → review+
(Assignee)

Comment 11

a year ago
(In reply to Kannan Vijayan [:djvj] from comment #10)
> Comment on attachment 8931107 [details]
> Bug 1374574 - Remove the FlyWeb service and DOM interfaces.
> 
> https://reviewboard.mozilla.org/r/202190/#review207890
> 
> Thank you for taking care of this.  Do you drink beer?  Are you coming to
> Austin?  Count me as owing you one beverage of your choice :)

I'm coming to Austin and I will happily accept that offer! :D

Comment 12

a year ago
mozreview-review
Comment on attachment 8931106 [details]
Bug 1374574 - Remove the FlyWeb system add-on from Fennec.

https://reviewboard.mozilla.org/r/202188/#review208498

LGTM. Make sure to run on try before submitting : ).
Attachment #8931106 - Flags: review+

Comment 13

a year ago
mozreview-review-reply
Comment on attachment 8931106 [details]
Bug 1374574 - Remove the FlyWeb system add-on from Fennec.

https://reviewboard.mozilla.org/r/202188/#review208498

Looks like I no longer have review powers, so you'll need to find another person. It might be worth trying maliu or nalexander.

Comment 14

a year ago
mozreview-review
Comment on attachment 8931106 [details]
Bug 1374574 - Remove the FlyWeb system add-on from Fennec.

https://reviewboard.mozilla.org/r/202188/#review208520
Attachment #8931106 - Flags: review?(s.kaspari) → review+

Comment 15

a year ago
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 9f773aae403d -d 50ea7f713c35: rebasing 435957:9f773aae403d "Bug 1374574 - Remove the FlyWeb system add-on from Firefox. r=mconley"
merging testing/talos/talos/mtio-whitelist.json
merging testing/talos/talos/xtalos/xperf_whitelist.json
warning: conflicts while merging testing/talos/talos/mtio-whitelist.json! (edit, then use 'hg resolve --mark')
warning: conflicts while merging testing/talos/talos/xtalos/xperf_whitelist.json! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 19

a year ago
Pushed by jhofmann@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/07ca590ac669
Remove the FlyWeb system add-on from Firefox. r=mconley
https://hg.mozilla.org/integration/autoland/rev/ff75ef015293
Remove the FlyWeb system add-on from Fennec. r=ahunt,sebastian
https://hg.mozilla.org/integration/autoland/rev/5a8a3d887e19
Remove the FlyWeb service and DOM interfaces. r=bz,djvj

Comment 20

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/07ca590ac669
https://hg.mozilla.org/mozilla-central/rev/ff75ef015293
https://hg.mozilla.org/mozilla-central/rev/5a8a3d887e19
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox59: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
status-firefox57: affected → ---

Updated

3 months ago
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.