1032121 - Remove leftover code from the removal of automatic content type handlers

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Runtime, defect)

Description

[Antonio Manuel Amaya Calvo (:amac)]

Bug 1009781 removed the activity -> Content type handler mapping but it left some code behind. That code is showing the error:

 "types.forEach is not a function" {file: "resource://gre/modules/ActivitiesService.jsm" line: 428}]

and generating an array that's then just throw out in the best case.

Comment 1

[Antonio Manuel Amaya Calvo (:amac)]

Attachment: V1. Remove code that should be dead

Although I don't expect any problems since what's removed just created an array that wasn't used anywhere, try run is at:
https://tbpl.mozilla.org/?tree=Try&rev=6b2cda0423ae

Comment 2

[[:fabrice] Fabrice Desré]

 https://hg.mozilla.org/integration/b2g-inbound/rev/c3b6129862d8

Comment 3

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c3b6129862d8

Comment 4

[Antonio Manuel Amaya Calvo (:amac)]

Comment on attachment 8447913 [details] [diff] [review]
V1. Remove code that should be dead

Approval Request Comment
[Feature/regressing bug #]: Bug 1009781 
[User impact if declined]: Unknown, it is failing on 2.0 and when it fails activities won't be completely registered or unregistered.
[Describe test coverage new/current, TBPL]: There are no new tests, this just removes leftover code from a previous change.
[Risks and why]: Very low, it just remove dead code (or code that should be dead)
[String/UUID change made/needed]: None

Comment 5

[Ryan VanderMeulen [:RyanVM]]

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