(In reply to Hsin-Yi Tsai (she/her) away [:hsinyi] from comment #3) > (In reply to Chris Peterson [:cpeterson] from comment #2) > > Hsin-Yi says this fix is needed for Android Fission. > > To be clearer, this blocks Android Fission if we plan to use the same the same [remoteTypePrefix](https://searchfox.org/mozilla-central/rev/1aa61dcd48e128a8cbfbe59b7ba43d31bd3c248a/dom/ipc/ContentChild.cpp#2687) as on desktop. If so, Spectre mitigation will be disabled automatically once Android Fission is enabled, regardless of ORB ( we don't check for ORB there currently). This would lead to unintended consequences, as ORB is a hard blocker preventing the removal of spectre mitigations. Leaving a comment here for clear reference: bug 1894265 ensures spectre mitigation remain disabled when ORB is not enabled on Android. So this bug isn't a blocker for Android Fission. >However, even though we will be using another remote type, it still makes sense to fix this issue as soon as possible after Android Fission, so we can disable spectre mitigations where it makes sense, thereby reducing maintenance complexity and gaining performance. This part is still true and important.
Bug 1778430 Comment 5 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
(In reply to Hsin-Yi Tsai (she/her) away [:hsinyi] from comment #3) > (In reply to Chris Peterson [:cpeterson] from comment #2) > > Hsin-Yi says this fix is needed for Android Fission. > > To be clearer, this blocks Android Fission if we plan to use the same the same [remoteTypePrefix](https://searchfox.org/mozilla-central/rev/1aa61dcd48e128a8cbfbe59b7ba43d31bd3c248a/dom/ipc/ContentChild.cpp#2687) as on desktop. If so, Spectre mitigation will be disabled automatically once Android Fission is enabled, regardless of ORB ( we don't check for ORB there currently). This would lead to unintended consequences, as ORB is a hard blocker preventing the removal of spectre mitigations. Leaving a comment here for clear reference: bug 1894265 ensures spectre mitigation remain enabled when ORB is not enabled on Android. So this bug isn't a blocker for Android Fission. >However, even though we will be using another remote type, it still makes sense to fix this issue as soon as possible after Android Fission, so we can disable spectre mitigations where it makes sense, thereby reducing maintenance complexity and gaining performance. This part is still true and important.
(In reply to Hsin-Yi Tsai (she/her) away [:hsinyi] from comment #3) > (In reply to Chris Peterson [:cpeterson] from comment #2) > > Hsin-Yi says this fix is needed for Android Fission. > > To be clearer, this blocks Android Fission if we plan to use the same the same [remoteTypePrefix](https://searchfox.org/mozilla-central/rev/1aa61dcd48e128a8cbfbe59b7ba43d31bd3c248a/dom/ipc/ContentChild.cpp#2687) as on desktop. If so, Spectre mitigation will be disabled automatically once Android Fission is enabled, regardless of ORB ( we don't check for ORB there currently). This would lead to unintended consequences, as ORB is a hard blocker preventing the removal of spectre mitigations. Leaving a comment here for clear reference: bug 1894265 ensures spectre mitigation remain enabled when ORB is not enabled on Android. So this bug is no longer a blocker for Android Fission. >However, even though we will be using another remote type, it still makes sense to fix this issue as soon as possible after Android Fission, so we can disable spectre mitigations where it makes sense, thereby reducing maintenance complexity and gaining performance. This part is still true and important.