| Auto-Submit | +1 |
| Commit-Queue | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
return _aimEligibilityService->IsAimEligible() &&nit: I think we can remove this since IsFuseboxEligible() checks IsAimEligible() as well. See: https://source.chromium.org/chromium/chromium/src/+/main:components/omnibox/browser/aim_eligibility_service.cc;l=534
aimAllowed = aimAllowed && _aimEligibilityService->IsFuseboxEligible();nit: same as above, not sure if we need to also check `aimAllowed`.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Auto-Submit | +1 |
| Commit-Queue | +1 |
nit: I think we can remove this since IsFuseboxEligible() checks IsAimEligible() as well. See: https://source.chromium.org/chromium/chromium/src/+/main:components/omnibox/browser/aim_eligibility_service.cc;l=534
Done
aimAllowed = aimAllowed && _aimEligibilityService->IsFuseboxEligible();nit: same as above, not sure if we need to also check `aimAllowed`.
aimAllowed also checks the form factor (we currently enable it separately on tablets), so I will keep this as is. But when we ship fusebox, we'll be able to remove the IsDisableComposeboxFromAIMNTPEnabled() check and then we can swap IsAimEligible with IsFuseboxEligible above.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | -1 |
| Commit-Queue | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
aimAllowed = aimAllowed && _aimEligibilityService->IsFuseboxEligible();This should not change AIM availability on the NTP it should redirect the AIM button to the web instead of fusebox.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Commit-Queue | +1 |
aimAllowed = aimAllowed && _aimEligibilityService->IsFuseboxEligible();This should not change AIM availability on the NTP it should redirect the AIM button to the web instead of fusebox.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
aimAllowed = aimAllowed && _aimEligibilityService->IsFuseboxEligible();Christian XuThis should not change AIM availability on the NTP it should redirect the AIM button to the web instead of fusebox.
This is handled at `openMIA` in the coordinator
I don't think it's correct. I think to have a correct comparison across experiment arms, this should be fully disabled for non-fusebox users. Let's confirm with PM.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Auto-Submit | +1 |
| Commit-Queue | +1 |
Prepping this alternative CL while waiting for PM decision on how to proceed.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Auto-Submit | +0 |
aimAllowed = aimAllowed && _aimEligibilityService->IsFuseboxEligible();Christian XuThis should not change AIM availability on the NTP it should redirect the AIM button to the web instead of fusebox.
Stepan KhapuginThis is handled at `openMIA` in the coordinator
I don't think it's correct. I think to have a correct comparison across experiment arms, this should be fully disabled for non-fusebox users. Let's confirm with PM.
Per offline discussion, even if we're going with what I think is the intention here, there should be an additional check for IsComposeboxIOSEnabled(). Added.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
return _aimEligibilityService->IsFuseboxEligible();Sorry for the churn here, but we should actually keep the `IsAimEligible` check in thie CL. Just found out that we will need to remove the `IsAimEligible` check in `IsFuseboxEligible` so keeping it here would be good to future proof this CL. Thank you!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
return _aimEligibilityService->IsFuseboxEligible();Sorry for the churn here, but we should actually keep the `IsAimEligible` check in thie CL. Just found out that we will need to remove the `IsAimEligible` check in `IsFuseboxEligible` so keeping it here would be good to future proof this CL. Thank you!
Nevermind confirmed with Moe that we can keep the mobile code as is. Please disregard the above message!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Commit-Queue | +2 |
lgtm!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
[iOS][aim-eligibility] Gate iOS fusebox on `IsFuseboxEligible`
1. Gate fusebox on fusebox eligibility
2. Redirect AIM NTP shortcut to web, depending on fusebox eligibility,
when it normally opens fusebox.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Stepan Khapugin abandoned this change.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |