851011 - Remove unused feed handlers from mobile/

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Frank Yan (:fryn)]

Attachment: patch: remove contentHandlers from mobile/

In bug 840710, Matt Brubeck informed me that mobile/ doesn't use contentHandlers (the feed handlers), so I'm removing them.

There are also gecko.handlerService prefs (protocol handlers for webcal and mailto). Simply by searching the codebase for "handlerService", I was not able to confidently deduce whether mobile/ actually used those. Matt, if we don't, I can remove those too.

Comment 1

[Matt Brubeck (:mbrubeck)]

Comment on attachment 724822 [details] [diff] [review]
patch: remove contentHandlers from mobile/

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

Looks good to me.  Possibly we could also use the related files from /mobile/android/installer/package-manifest.in.  Adding mfinkle just to double-check this patch.

I have no idea whether the handlerService prefs are used on Android; if someone wants to investigate that we could file a separate bug.

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 724822 [details] [diff] [review]
patch: remove contentHandlers from mobile/

With the news of Google Reader being killed and new features we might be adding to Firefox's Reader Mode, I'd like to keep these for now.

We might even revive some old patches to add Feed support to Fennec.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

WONTFIX'ing for now

Comment 4

[Frank Yan (:fryn)]

(In reply to Mark Finkle (:mfinkle) from comment #2)
> Comment on attachment 724822 [details] [diff] [review]
> patch: remove contentHandlers from mobile/
> 
> With the news of Google Reader being killed and new features we might be
> adding to Firefox's Reader Mode, I'd like to keep these for now.

Why are we keeping the URL for adding to Reader/iGoogle when iGoogle is already discontinued for mobile and Reader will be discontinued in July? Removing these handler URLs has no negative future effect on Firefox for Android's ability to support feed viewing/subscription.

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

If you really want to remove the preferences, I'd be OK with that. Keep the py files as is, so if we add preferences back, things just work without pulling out too much hair.

Comment 6

[Frank Yan (:fryn)]

Attachment: patch: just remove the prefs

I'd like to remove the prefs, because they are unused and contain non-SSL links, and they are being caught in the current sweep to move all our in-product URLs to https:// (SSL).

If/when we add subscription support back in, I highly doubt that Add to My Yahoo! in its current form will be the top choice of the future, especially on mobile.

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 725137 [details] [diff] [review]
patch: just remove the prefs

thanks

Comment 8

[Frank Yan (:fryn)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/0a289854fe9d

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/0a289854fe9d