diff options
author | maxwelb <maxwelb@google.com> | 2017-07-19 16:25:50 -0700 |
---|---|---|
committer | Eric Erfanian <erfanian@google.com> | 2017-07-25 16:43:24 +0000 |
commit | 37768b95c107421320b7fba30bb81158b8215e2f (patch) | |
tree | de16bae5490130a13a74a00a59259ee7c58ae008 /java/com/android/dialer/callcomposer/camera/ImagePersistTask.java | |
parent | 0c408e0e3c82bff80871c3a3f2de5d37bc398a42 (diff) |
Fix non-normalized number bug with FuzzyPhoneNumberMatcher
This CL fixes a bug with the FuzzyPhoneNumberMatcher implementation. The
logic should be that if the last 7 digits of two numbers match, we treat
them as the same. The issue was that if we found digits that didn't
match between the number, we reported them as not matching, even if they
had 7 digits in common.
This CL also makes some quality of life changes:
- Renames FuzzyNumberMatcher to FuzzyPhoneNumberMatcher to match the
class name.
- Abstracts the logic of the FuzzyPhoneNumberMatcher into another well
named method, so it's more clear how the code works.
While I didn't check too carefully, I believe this will also help out
issues with capability flakiness in:
- b/62609419 | maxwelb | RCS very unstable when on cellular.
Bug: 63564400,62609419
Test: FuzzyPhoneNumberMatcherTest
PiperOrigin-RevId: 162552340
Change-Id: Ie091d985a34a16000750f6050bde6bb0ea7da4e5
Diffstat (limited to 'java/com/android/dialer/callcomposer/camera/ImagePersistTask.java')
0 files changed, 0 insertions, 0 deletions