• Regan Hsu's avatar
    Reland "[CrOS PhoneHub] Mark tether ineligible if SIM has no reception." · 7a2bf75e
    Regan Hsu authored
    This is a reland of a42b2a30.
    
    phone_model_test_util's CreateFakePhoneStatusModel() returned
    unexpected mobile status, resulting in failing tests. Created
    TetherControllerImplTest's own CreateFakePhoneStatusModel() to fix.
    
    Original change's description:
    > [CrOS PhoneHub] Mark tether ineligible if SIM has no reception.
    >
    > TetherControllerImpl will now mark the tether feature ineligible if
    > it is notified by the PhoneModel that the SIM state has changed to
    > anything other than kSimWithReception.
    >
    > Fixed: 1140812
    > Bug: 1106937
    > Change-Id: Ia18c5e06c57805541b03b29afea1762e85b8304f
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2500182
    > Commit-Queue: Regan Hsu <hsuregan@chromium.org>
    > Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#821493}
    
    Fixed: 1140812
    Fixed: 1143173
    Bug: 1106937
    Change-Id: I310daa1d1feea6a74485ddd143608b216414f3bb
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2507632
    Commit-Queue: Regan Hsu <hsuregan@chromium.org>
    Reviewed-by: default avatarKyle Horimoto <khorimoto@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#822892}
    7a2bf75e
phone_hub_manager_impl.cc 6.75 KB