On Mac, there is no IEProfileMigrator.js. On Linux, there is neither. Yet, in both builds, they both are in the components manifest.

Comment on attachment 618955 [details] [diff] [review] Only reference IE and Safari migrators in components manifest when they are installed Review of attachment 618955 [details] [diff] [review]: ----------------------------------------------------------------- should be safe to backport to 13 too, so please get approval and do :)

The commit message is misleading because we can't detect "when they are installed" at build time. Something like this would be better: "Only reference IE and Safari migrators in components manifest on supported operating systems."

It's "installed" in the "$(NSINSTALL)" sense.

(In reply to Marco Bonardo [:mak] from comment #2) > should be safe to backport to 13 too, so please get approval and do :) I actually meant "backport to 14"... Merge didn't help my numbering.

Comment on attachment 618955 [details] [diff] [review] Only reference IE and Safari migrators in components manifest when they are installed [Approval Request Comment] Regression caused by: bug 710259 and bug 710895 User impact if declined: No real user impact, but Firefox uselessly tries to registers components that aren't there on Mac and Linux. Testing completed (on m-c, etc.): Landed on m-c a couple days ago. Risk to taking this patch (and alternatives if risky): None String changes made by this patch: None

Comment on attachment 618955 [details] [diff] [review] Only reference IE and Safari migrators in components manifest when they are installed [Triage Comment] We don't typically take correctness fixes with no user impact.