{"push_id":693372381,"size":1,"distinct_size":1,"ref":"refs/heads/sunset","head":"41db8bcf797b6ceb5a159e156a572afd0cf0877d","before":"901ae3f2f0bc8fb00dfa92509382fb761dad80a9","commits":[{"sha":"41db8bcf797b6ceb5a159e156a572afd0cf0877d","author":{"email":"638edf137583a01a6035486ebbcb61eccfc36ade@chromium.org","name":"Martin Sramek"},"message":"Remove the check for large notification index.\n\nIn shouldShowNotification_, we immediately return true for large indices. This is the correct behavior, since it allows us to try again if the extension uninstallation failed.\n\nHowever, in maybeShowNotification_, we set the index to zero if it is out of range, which means that we reset the timeline. This CL removes the upper bound check in maybeShowNotification_, so that this doesn't happen.\n\nTo test that the whole process works, we add a mock of Chrome extension APIs.","distinct":true,"url":"https://api.github.com/repos/msramek/chrome-opt-out-extension/commits/41db8bcf797b6ceb5a159e156a572afd0cf0877d"}]}